-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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: store column IDs in index predicates instead of column names #49766
Comments
Some things that I've been talking over with @RichardJCai in this regard:
There may be other gotchas. Anything else you noticed, Richard? |
No I think you've covered everything. I don't think table id will be too difficult to update right? |
We have marked this issue as stale because it has been inactive for |
Storing column names in partial index predicates (like
CREATE INDEX ... WHERE s = 'foo'
) has disadvantages, such as maintaining code to handle column renames.Storing column IDs instead (like `CREATE INDEX ... WHERE @11 = 'foo') would eliminate this issue.
Note that we'd need to convert these column IDs to column names whenever the predicate is shown to users.
More context: #49672 (comment)
Jira issue: CRDB-4199
The text was updated successfully, but these errors were encountered: