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

reformat codec and add keyspace support #649

Merged
merged 13 commits into from
Jan 10, 2023

Conversation

AmoebaProtozoa
Copy link
Contributor

@AmoebaProtozoa AmoebaProtozoa commented Dec 27, 2022

This PR refactors CodecPDClient, CodecPDClientV2 and api version into a unified codec interface that supports encode/decode request under multiple operation mode.

issue: #545

Signed-off-by: David 8039876+AmoebaProtozoa@users.noreply.github.com

AmoebaProtozoa and others added 10 commits December 27, 2022 10:08
Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
…pace-utils

Signed-off-by: iosmanthus <myosmanthustree@gmail.com>
add KeyspaceID parsing utils for codec_v2
Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
@AmoebaProtozoa AmoebaProtozoa changed the title [WIP] reformat codec and add keyspace support reformat codec and add keyspace support Jan 6, 2023
@AmoebaProtozoa AmoebaProtozoa marked this pull request as ready for review January 6, 2023 07:30
internal/apicodec/codec.go Outdated Show resolved Hide resolved
Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
Signed-off-by: David <8039876+AmoebaProtozoa@users.noreply.github.com>
@ystaticy
Copy link
Contributor

ystaticy commented Jan 6, 2023

LGTM

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.

4 participants