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.

Sign up for GitHub

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

Jump to bottom

Fix JSONPath inconsistencies in DiceDB #1002

Open
JyotinderSingh opened this issue Oct 7, 2024 · 6 comments · May be fixed by #1266
Open

Fix JSONPath inconsistencies in DiceDB #1002

JyotinderSingh opened this issue Oct 7, 2024 · 6 comments · May be fixed by #1266
Assignees
@N3XT14
Labels
difficulty-level -- hard stack -- go

Comments

@JyotinderSingh
Copy link
Collaborator

Reference: #493 (comment)

Response Comparison

Data Link

As you can see, there are several inconsistencies between redis and dice responses. This is largely because the JSONPath library DiceDB uses does not cover the entire redis JSONPath feature set.
@N3XT14
Copy link

N3XT14 commented Oct 7, 2024

Would love to work on this? .
Thanks

@lucifercr07
Copy link
Contributor

@N3XT14 assigned, thanks for contributing.

@Lockps Lockps mentioned this issue Oct 13, 2024
Open
@surya0180
Copy link
Contributor

@N3XT14 Any update on this issue? If in case you are not able to work on this, I am willing to work on this issue.

@lucifercr07 @JyotinderSingh FYI

@N3XT14
Copy link

N3XT14 commented Oct 16, 2024

@surya0180 Yeah working on it. implementing the necessary things

@N3XT14
Copy link

N3XT14 commented Oct 21, 2024

Quick Update:

I am progressing on it. Currently making it work between resp2 & resp3

@N3XT14
Copy link

N3XT14 commented Nov 11, 2024

@JyotinderSingh #1266

Not ready for merge but would like to get some questions answered regarding RESP3 and other output structures.

@JyotinderSingh JyotinderSingh linked a pull request Nov 11, 2024 that will close this issue
Open
@apoorvyadav1111 apoorvyadav1111 mentioned this issue Nov 14, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@N3XT14 N3XT14

Labels
difficulty-level -- hard stack -- go
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix JSONPath inconsistencies N3XT14/dice
4 participants
@lucifercr07 @JyotinderSingh @N3XT14 @surya0180