Skip to content

refactor: replace byteorder with SketchBytes and SketchSlice #49

Merged
leerho merged 5 commits into
apache:mainfrom
tisonkun:codec
Dec 30, 2025
Merged

refactor: replace byteorder with SketchBytes and SketchSlice #49
leerho merged 5 commits into
apache:mainfrom
tisonkun:codec

Conversation

@tisonkun

Copy link
Copy Markdown
Member

This closes #41.

I'd wait for a review from @notfilippo @Xuanwo

cc @PsiACE @ZENOTME

Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>
Signed-off-by: tison <wander4096@gmail.com>

@ZENOTME ZENOTME left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@leerho leerho merged commit b0e0135 into apache:main Dec 30, 2025
9 checks passed
@tisonkun tisonkun deleted the codec branch December 30, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Encapsulate codec over Vec<u8> and slices

3 participants