-
Notifications
You must be signed in to change notification settings - Fork 433
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 the Redis JSON support #1805
Labels
Comments
PragmaTwice
changed the title
Tracking issue for Redis JSON support
Tracking issue for the Redis JSON support
Oct 15, 2023
This was referenced Oct 15, 2023
I found that some of the commands were marked as deprecated in the Redis document, do you guys still want to support it? |
@Kelvinyu1117 I think we can complete the active commands first and the deprecated commands are also good to support if users did have solid reasons. cc @PragmaTwice |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Motivation
After #1803, we have basic support of RedisJSON, but massive amount of commands remain unsupported:
We need to support all of these commands to make our JSON support more complete and powerful.
Other pull requests related to JSON:
JSON.GET
#1840Call for Contribution
Feel free to pick up any issue and finish it!
You can request to be assigned with any issue from this list and try to implement it.
How to implement
Parse
andExecute
function) to commands/cmd_json.cc. (Note:types/json.h
is just wrapper functions ofjsoncons
library which is not related to storage, whiletypes/redis_json.h
is about how to encode/decode JSON from storage (IO operations))The text was updated successfully, but these errors were encountered: