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

sql,spanconfig: temp indexes for index backfill should share a configuration with the final index #76608

Closed
ajwerner opened this issue Feb 15, 2022 · 1 comment
Labels
A-disaster-recovery branch-master Failures and bugs on the master branch. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) GA-blocker T-disaster-recovery

Comments

@ajwerner
Copy link
Contributor

ajwerner commented Feb 15, 2022

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

As described in #70426 / RFC and implemented in #73878, we'll soon create a new "temporary" index which will be written to during index backfills. Users can configure zone configurations for indexes prior to backfilling them (this can be useful). Any zone configurations applied to an index should also be applied to its temp index.

Describe the solution you'd like

We don't create these temp indexes until after span configs are in use. In principle, it shouldn't be too hard to pair a temp index with the backfilling index (I think we need a better API, but it shouldn't be hard). I'd like the sqltranslator to apply the same config to the temp index that it applies to its corresponding backfilling index. This should be easy.

Jira issue: CRDB-13192

@ajwerner ajwerner added the C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) label Feb 15, 2022
@ajwerner ajwerner added branch-master Failures and bugs on the master branch. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. labels Feb 15, 2022
@blathers-crl
Copy link

blathers-crl bot commented Mar 8, 2022

cc @cockroachdb/bulk-io

@dt dt added GA-blocker and removed release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. labels Mar 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-disaster-recovery branch-master Failures and bugs on the master branch. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) GA-blocker T-disaster-recovery
Projects
No open projects
Archived in project
Development

No branches or pull requests

4 participants