Skip to content

feat(gms): add primary storage read pools for Ebean and Cassandra#17719

Open
david-leifker wants to merge 2 commits into
masterfrom
feat/primary-storage-read-pool
Open

feat(gms): add primary storage read pools for Ebean and Cassandra#17719
david-leifker wants to merge 2 commits into
masterfrom
feat/primary-storage-read-pool

Conversation

@david-leifker
Copy link
Copy Markdown
Collaborator

Route non-locking aspect reads to optional read pools with OperationContext read preference, JDBC read-only connections for Postgres/MySQL, and docs plus Testcontainers integration tests for split-pool routing.

@github-actions github-actions Bot added docs Issues and Improvements to docs product PR or Issue related to the DataHub UI/UX devops PR or Issue related to DataHub backend & deployment labels Jun 3, 2026
Route non-locking aspect reads to optional read pools with OperationContext
read preference, JDBC read-only connections for Postgres/MySQL, and docs plus
Testcontainers integration tests for split-pool routing.

Co-authored-by: Cursor <cursoragent@cursor.com>
@david-leifker david-leifker force-pushed the feat/primary-storage-read-pool branch from 5e20576 to 37f33eb Compare June 3, 2026 20:18
@maggiehays maggiehays added the needs-review Label for PRs that need review from a maintainer. label Jun 3, 2026
Add unit tests for PrimaryStorageResolver, registry, context, factories,
and Cassandra read-pool session wiring to meet Codecov patch thresholds.

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

devops PR or Issue related to DataHub backend & deployment docs Issues and Improvements to docs needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants