-
Notifications
You must be signed in to change notification settings - Fork 4.1k
kv,server: enable some degree of range descriptor pre-fetching in secondary tenants #91433
Copy link
Copy link
Closed
Labels
C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)
Description
Is your feature request related to a problem? Please describe.
Currently we do not pre-fetch any additional range descriptors. See
cockroach/pkg/ccl/kvccl/kvtenantccl/connector.go
Lines 397 to 400 in 4488445
| // Until we add protection in the Internal service implementation to | |
| // prevent prefetching from traversing into RangeDescriptors owned by | |
| // other tenants, we must disable prefetching. | |
| PrefetchNum: 0, |
Describe the solution you'd like
We should add some logic to filter range descriptors returned based on the tenant and then use the same pre-fetching logic we use in regular KV.
Epic: CRDB-18596
Jira issue: CRDB-21259
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
C-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)