[DOC-1360][2025.2.4] Document ALTER ROLE/ALTER DATABASE handling in YSQL Connection Manager#32138
[DOC-1360][2025.2.4] Document ALTER ROLE/ALTER DATABASE handling in YSQL Connection Manager#32138arpit-saxena wants to merge 6 commits into
Conversation
The section heading was renamed to 'ALTER ROLE SET / ALTER DATABASE SET commands', so update the two in-page links to use the matching blackfriday anchor and link text. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the YSQL Connection Manager documentation to introduce two new configuration flags, ysql_conn_mgr_alter_guc_adoption_strategy and ysql_conn_mgr_alter_guc_stale_backend_ttl_ms, which manage how GUC settings changed by ALTER ROLE/DATABASE statements are adopted. It also documents a known limitation regarding the propagation of these changes across multiple nodes. The review feedback suggests replacing the slash in the new markdown header with 'and' to ensure consistent anchor generation and prevent broken links across different markdown renderers.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| - Any string-type variables of extensions | ||
| - `yb_read_after_commit_visibility` | ||
|
|
||
| ## ALTER ROLE SET / ALTER DATABASE SET commands |
There was a problem hiding this comment.
Using a slash (/) in markdown headers can lead to inconsistent anchor generation across different markdown renderers (such as GitHub vs. Hugo/Goldmark), which can break relative links. Consider replacing the slash with and to ensure consistent and robust anchor generation. If you apply this change, please also update the corresponding links on lines 127 and 264 to use #alter-role-and-alter-database-set-commands.
| ## ALTER ROLE SET / ALTER DATABASE SET commands | |
| ## ALTER ROLE and ALTER DATABASE SET commands |
There was a problem hiding this comment.
Done in e48585e. Replaced the slash with and in the header and updated both cross-references (lines 127 and 264) to the new anchor #alter-role-set-and-alter-database-set-commands. Note the header has SET in both halves, so the anchor includes it.
automated · Claude Code (Opus 4.8)
✅ Deploy Preview for infallible-bardeen-164bc9 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Slashes in markdown headers can produce inconsistent anchors across renderers (GitHub vs. Hugo/Goldmark). Update the header and the two cross-references to use the slash-free anchor. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2 participants
This doc change documents how ALTER ROLE SET and ALTER DATABASE SET commands are handled by YSQL Connection Manager, including the new ysql_conn_mgr_alter_guc_adoption_strategy and ysql_conn_mgr_alter_guc_stale_backend_ttl_ms flags.