Avoid check/assign hooks of conn mgr guc variables for parallel worker #23787
Assignees
Labels
Comments
yugabyte-ci
added
jira-originated
kind/bug
manav-yb
pushed a commit
that referenced
this issue
Sep 5, 2024
…lel workers Summary: This diff avoids executing check/assign hooks of ysql connection manager specific guc variables while initialising the parallel background worker. **Reason for crash** The hooks for connection manager specific guc variables `yb_is_client_ysqlconnmgr`, `yb_use_tserver_key_auth` were failing for parallel background workers. These guc variables are set by ysql connection manager while creating the physical connections and expects connections to be of unix socket type. Whereas for parallel background workers created by postmaster it's not necessary nor these guc variables are applicable. Therefore avoiding the hooks. This diff fixes the tests mentioned in test plan which used to fail due to same above reason. Tests are failing only with tsan build but not due to this diff/reason, it's been tracked here [[ https://yugabyte.atlassian.net/browse/DB-12641 | DB-12641 ]] Jira: DB-12690 Test Plan: Jenkins: enable connection manager, test regex: .*PgParallelReadIsolation.*|.*PgTransparentRestarts.* Reviewers: skumar, asrinivasan, stiwary, devansh.saxena, rbarigidad Reviewed By: skumar, stiwary Subscribers: yql Differential Revision: https://phorge.dev.yugabyte.com/D37767
jasonyb
pushed a commit
that referenced
this issue
Sep 6, 2024
Summary: c587efd [docs] minor edit (#23796) 31e09f3 [PLAT-15029] yba installer split data and software directory setup f5ba17d [PLAT-15039]: Fix bootstrap on bi-directional xCluster config creation 578248a [#23770] YSQL: Deterministically populate catalog cache in tests with Connection Manager enabled 7b1f22a [#23799] test: Fixed PgTableSizeTest.SharedTableSize test for pg15 788434a [#18771, #21352] docdb: Fix LightweightMessage max size when parsing 02ced43 [#22821] YSQL: Preserve local limit in a multi-page read 50ff737 [#23741] docdb: Fix cloning of colocated databases with only parent table Excluded: 9889df7 [#23706] YSQL: Add table-level catcache Prometheus metrics c770d79 [#23747] MetaCache: Callback should not be called while holding the lock Excluded: 40689bc [#22150] YSQL, QueryDiagnostics: EXPLAIN (ANALYZE, DIST) support for queryDiagnostics 1655e69 [PLAT-15148]: Set XCluster Table Status to DroppedFromTarget if table in replication is dropped from target only 16262f7 [#22519] YSQL: Simplify API of the ExplicitRowLockBuffer class 6614afb [PLAT-14958][PLAT-14959] Make ssh fields optional if skipProvisioning is true 1153b56 [PLAT-14867] Make sure restart alerts don't trigger for small time updates during NTP sync bf1c7bc [PLAT-12226] Add connection pooling status to universe health check 38d8ae8 [PLAT-14805]Support adding EAR configs a180bef [#19134] YSQL, ASH: Setting ASH circular buffer size based on the number of cores 7d8fc76 Adjust heading link (#23807) 4c6cf5a [PLAT-4899]Basic validation of certificates f24eb10 [#23787] YSQL: Avoid executing conn mgr guc variables hooks for parallel workers ee18df8 [PLAT-13921] [K8] [UI] Universe action tasks are disabled after a failed shrink rr node task Excluded: dcf1821 [#23797] YSQL: Modify some tests to run in single connection mode with Connection Manager 0ac22cd [Docs] Remove Drift chat bot (#23802) 0e91003 [#22825] DocDB: Vector Index General Read Path with DummyANN e8f09b5 [PLAT-15175] Make runtime conf for skipping cluster consistency check public ee479ee Versionwarning (#23781) a05c6a3 [DB-12681] yugabyted-ui: Add Voyager commands to different Voyager phases in the UI. cc80d59 [#23777] yugabyted: updating the pg parity testcase to reflect the new gflags enabled for the pg parity feature. Test Plan: Jenkins: rebase: pg15-cherrypicks Reviewers: jason, tfoucher Subscribers: yql Differential Revision: https://phorge.dev.yugabyte.com/D37822
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Jira Link: DB-12690
The text was updated successfully, but these errors were encountered: