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: maintain and allocate from an index ID freelist on tables #52926

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rohany
Copy link
Contributor

@rohany rohany commented Aug 17, 2020

This commit adds an ID freelist data structure and uses it to allocate
index ID's for new indexes on tables.

Release note: None

This commit adds an ID freelist data structure and uses it to allocate
index ID's for new indexes on tables.

Release note: None
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rohany
Copy link
Contributor Author

rohany commented Aug 17, 2020

This would need a good bit more polish before landing, but I wanted to put it up to see what the thoughts are. It also needs something that @dt is working on (i think?) that ensures that we wait until a clearrange's application deadline exceeds before adding the index ID to the free list.

@thoszhang thoszhang removed their request for review February 26, 2021 22:45
@tbg tbg added the X-noremind Bots won't notify about PRs with X-noremind label May 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
X-noremind Bots won't notify about PRs with X-noremind
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants