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

Tracking issue for dependency redis-protocol #2

Open
J0HN50N133 opened this issue Apr 4, 2024 · 0 comments
Open

Tracking issue for dependency redis-protocol #2

J0HN50N133 opened this issue Apr 4, 2024 · 0 comments

Comments

@J0HN50N133
Copy link
Collaborator

Currently, in redis-protocol, if you enable codec and disable resp2, resp2 will still be enabled since codec implies resp2. However, there is no reason to enable this if I've already disabled resp2. Currently, use the redis-protocol of my own fork.

J0HN50N133 added a commit that referenced this issue Apr 4, 2024
J0HN50N133 added a commit that referenced this issue Apr 10, 2024
J0HN50N133 added a commit that referenced this issue Apr 11, 2024
* feat: add commands crate

* fix: ut log level should at least 'debug'

* chore(deps): vendor redis-protocol to my own fork

as describe in #2

* feat: add commands parser framework

* chore(deps): use redis-protocol 5.0.1

* refactor: database hold borrow directly instead of Arc to avoid overhead

* refactor: use serde to deserialize config type instead of builder

* feat: add execution process of set command

* feat: read configuration file

* feat: support parse ttl

* refactor: clean the implementation of Set::execute

* test: support test in ramdisk
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

No branches or pull requests

1 participant