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

Remove zero padding from the key in the range index #103

Closed
raminqaf opened this issue Oct 11, 2022 · 0 comments · Fixed by #101
Closed

Remove zero padding from the key in the range index #103

raminqaf opened this issue Oct 11, 2022 · 0 comments · Fixed by #101
Assignees
Labels

Comments

@raminqaf
Copy link
Contributor

Currently, we are padding the key of the topic with zeros and creating the range index. This is not needed. It is sufficient to zero-pad the value and concatenate the key with the value for the range index.
This will remove the type limitation of the key. Now the key should be only an integer or long. But after removing the zero padding, the key can also be a string.

@raminqaf raminqaf added this to the Complex GraphQL queries milestone Oct 11, 2022
@raminqaf raminqaf self-assigned this Oct 11, 2022
@raminqaf raminqaf added this to Quick Oct 11, 2022
@raminqaf raminqaf moved this to In Progress in Quick Oct 11, 2022
@raminqaf raminqaf moved this from In Progress to In-Review in Quick Oct 12, 2022
raminqaf added a commit that referenced this issue Oct 12, 2022
Repository owner moved this from In-Review to Done in Quick Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant