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

Error returned with redis logline #7

Open
arpit1912 opened this issue Oct 26, 2024 · 7 comments
Open

Error returned with redis logline #7

arpit1912 opened this issue Oct 26, 2024 · 7 comments

Comments

@arpit1912
Copy link

While running any wrong command, the CLI return back logline with redis in it. For example:

dicedb> SET k1 apple
OK
dicedb> GET k2
Error: redis: nil
dicedb> GET k1
apple

Expected behaviour:

It should have diceDB in the naming instead.

@Pratham-B-Parlecha
Copy link

Pratham-B-Parlecha commented Nov 1, 2024

Hey @arpit1912 can you assign this issue to me

@roopeshsn
Copy link

I think this should be handled here: https://github.com/DiceDB/dicedb-go

@JyotinderSingh
Copy link
Collaborator

I think this should be handled here: https://github.com/DiceDB/dicedb-go

That's correct. @Pratham-B-Parlecha Would you be interested in raising a fix for this issue on the dicedb-go repository? If not we can unassign this issue.

@Pratham-B-Parlecha
Copy link

@JyotinderSingh im interested in fixing this error on dicedb-go

@JyotinderSingh
Copy link
Collaborator

JyotinderSingh commented Nov 5, 2024

@JyotinderSingh im interested in fixing this error on dicedb-go

Awesome, please create a bug for this on the SDK repo. I'll assign it to you.

@Pratham-B-Parlecha
Copy link

Hi @JyotinderSingh, could you please create the bug and assign it to me? I am unable to create the bug myself.
Thank you!

@JyotinderSingh
Copy link
Collaborator

Hi @JyotinderSingh, could you please create the bug and assign it to me? I am unable to create the bug myself.

Thank you!

Please start working on the fix in your fork and raise a PR. I'll try to figure out the permission issue in the meantime.

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

4 participants