-
Notifications
You must be signed in to change notification settings - Fork 23
119-Introduce-CompactedKeyEncoder #124
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
Merged
luoyuxia
merged 37 commits into
apache:main
from
leekeiabstraction:119-Introduce-CompactedKeyEncoder
Jan 9, 2026
Merged
Changes from 23 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
0350ba1
Introduce CompactedKeyEncoder, initial commit
leekeiabstraction a02719a
Introduce CompactedKeyEncoder: working test cases
leekeiabstraction bf9215d
Use Datum, remove Value
leekeiabstraction 111b08f
All data type unit test
leekeiabstraction 082257d
Use Result in KeyEncoder
leekeiabstraction f506ab0
Update todo comment
leekeiabstraction b6e308e
Mark test methods as cfg(test)
leekeiabstraction a71db15
ValueWriter documentation
leekeiabstraction 006d593
Minor refactoring
leekeiabstraction 7dcea05
Add null check
leekeiabstraction a6c26a6
Improve todo message
leekeiabstraction fe7da7b
Move licence to top
leekeiabstraction 93ca8ab
More readable test case
leekeiabstraction c668dc5
Improve error message, use write_bytes for BytesWriter
leekeiabstraction ad3bd4a
Fix documentation
leekeiabstraction ac1d4b3
Use Result<> to return CompactedKeyEncoder and ValueWriter for better…
leekeiabstraction f29e2cc
More idiomatic implementation of encode_key
leekeiabstraction cbd0e0f
Improve documentation
leekeiabstraction 352f157
Improve error message
leekeiabstraction 16f4514
Improve error message
leekeiabstraction 07d6105
Minor refactor
leekeiabstraction 145830a
Formatting and clippy
leekeiabstraction e1d85a6
Formatting and clippy
leekeiabstraction 1ec7693
Addressed PR comments
leekeiabstraction 700bb47
Addressed PR comments
leekeiabstraction ca4036c
Improve error message
leekeiabstraction 4df3a81
Clippy
leekeiabstraction bcb7e08
Improve error message
leekeiabstraction b496612
Improve error message
leekeiabstraction 043e1d4
Improve and remove duplicate todos
leekeiabstraction 0ac34be
Fix test
leekeiabstraction 88ec810
More succinct code
leekeiabstraction d5a916b
More succinct code
leekeiabstraction eb70020
Use static dispatch for more performant code
leekeiabstraction eff7a88
Use static dispatch for better performance
leekeiabstraction b02233a
Move for_test_row_type function into test module
leekeiabstraction 3f5b938
More idiomatic row type building
leekeiabstraction File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.