Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

settings: utilize LOCALITY GLOBAL for cluster settings #85738

Open
ajwerner opened this issue Aug 8, 2022 · 3 comments
Open

settings: utilize LOCALITY GLOBAL for cluster settings #85738

ajwerner opened this issue Aug 8, 2022 · 3 comments
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Comments

@ajwerner
Copy link
Contributor

ajwerner commented Aug 8, 2022

Is your feature request related to a problem? Please describe.

Accessing cluster settings is important during cluster startup.

Describe the solution you'd like

The ideal tool to reduce the latency of reading cluster settings would be to set the system.settings table to use LOCALITY GLOBAL.

Describe alternatives you've considered
Use stale reads and reason about any issues which might arise. The biggest problems seem related to cluster version potentially moving backwards. Perhaps we could work around this by waiting out some wall-clock time period when setting the version. This doesn't seem worth the complexity if we make it possible to set some system tables to LOCALITY GLOBAL.

Additional context
This will depend on #63365.

Jira issue: CRDB-18412

@ajwerner ajwerner added the C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) label Aug 8, 2022
@exalate-issue-sync exalate-issue-sync bot changed the title settings: utilize LOCALITY GLOBAL for cluster settings settings: utilize LOCALITY GLOBAL for cluster settings for serverless Apr 11, 2023
@exalate-issue-sync exalate-issue-sync bot added the T-sql-schema-deprecated Use T-sql-foundations instead label Apr 11, 2023
@exalate-issue-sync exalate-issue-sync bot changed the title settings: utilize LOCALITY GLOBAL for cluster settings for serverless settings: utilize LOCALITY GLOBAL for cluster settings Apr 11, 2023
@exalate-issue-sync exalate-issue-sync bot added T-serverless and removed T-sql-schema-deprecated Use T-sql-foundations instead labels Apr 25, 2023
@rafiss
Copy link
Collaborator

rafiss commented Apr 25, 2023

cc @jeffswenson we re-discovered this issue during backlog grooming in SQL Schema. is this on your radar?

@jeffswenson
Copy link
Collaborator

This is not an issue for Serverless. When a tenant db is configured as multi-region, it configures the tenant settings as global. When CC automation creates a multi-region storage cluster, it uses zone configs to enable global reads on the storage cluster's settings table.

@rafiss
Copy link
Collaborator

rafiss commented Apr 25, 2023

Thanks for explaining! Then I'll leave this in our backlog for now until we see the priority come up in a different way.

@exalate-issue-sync exalate-issue-sync bot added T-sql-schema-deprecated Use T-sql-foundations instead and removed T-serverless labels Apr 25, 2023
@healthy-pod healthy-pod added T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) and removed T-sql-schema-deprecated Use T-sql-foundations instead labels May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)
Projects
None yet
Development

No branches or pull requests

4 participants