-
Notifications
You must be signed in to change notification settings - Fork 177
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
add direct
indices, except for in datastore & C#
#2205
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the schema stuff looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
macro/bindings stuff generally LGTM; UniqueColumn
will probably need an extra bound in the future in order to tell it whether to use btree or direct, but that's fine.
086b62f
to
70bdde4
Compare
ef7ddf6
to
5aae1d8
Compare
Description of Changes
Adds the schema and rust module side support for
direct
indices.What remains to be done after this PR is to:
API and ABI breaking changes
None
Expected complexity level and risk
3, touches a lot of places.
Testing
Some new tests are added for the schema and existing tests are amended.