Skip to content

Add deep analysis doc for client-side session reuse and index it under analysis#501

Draft
Copilot wants to merge 1 commit into
mainfrom
copilot/analyze-session-reuse-options
Draft

Add deep analysis doc for client-side session reuse and index it under analysis#501
Copilot wants to merge 1 commit into
mainfrom
copilot/analyze-session-reuse-options

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 10, 2026

This PR documents what is required to support client-side persistent JDBC session reuse in OJP (avoiding connect/terminateSession per-request overhead), including constraints around bounded retention and queued/in-flight work awareness.
It captures architectural impact, rollout guidance, and risk boundaries so implementation can proceed from a clear decision baseline.

  • New analysis document

    • Added documents/analysis/CLIENT_SIDE_SESSION_REUSE_ANALYSIS.md.
    • Covers:
      • current verified lifecycle behavior (connect fast path vs close -> terminateSession)
      • required driver/server changes (lease manager, close semantics split, sanitization, limits, observability)
      • multinode/XA scope boundaries
      • risks, phased rollout, and explicit opinions/suggestions/questions/concerns.
  • Analysis index update

    • Updated documents/analysis/README.md to include the new document under a dedicated Session Lifecycle & Reuse section.
    • Updated the index “Last Updated” date.

Example index entry added:

### Session Lifecycle & Reuse

- [CLIENT_SIDE_SESSION_REUSE_ANALYSIS.md](./CLIENT_SIDE_SESSION_REUSE_ANALYSIS.md) - Analysis of client-side persistent session reuse, required architecture changes, limits, risks, and rollout approach

@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants