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

Feature Request: Allow setting the hashing used for lookup vindexes in CreateLookupVindex #14155

Closed
Tracked by #13960
dbussink opened this issue Oct 2, 2023 · 0 comments · Fixed by #16113
Closed
Tracked by #13960

Comments

@dbussink
Copy link
Contributor

dbussink commented Oct 2, 2023

Feature Description

The CreateLookupVindex command does not allow overriding the hashing used for a lookup vindex on the backing lookup tables.

Use Case(s)

In #13955 we changed the default for integral types, but the same issue exists for string or binary types (which use MD5 based hashing by default atm, although to in a security context).

We could look at overriding the default, but in order to be safely migrate we also need to be able to have CreateLookupVindex support changing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment