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

[Feature]-[42]:[Redis Plugin] Add support for response type of List<List> #6185

Open
trishaanand opened this issue Jul 27, 2021 · 1 comment
Open

[Feature]-[42]:[Redis Plugin] Add support for response type of List<List> #6185

trishaanand opened this issue Jul 27, 2021 · 1 comment
Labels
Backend This marks the issue or pull request to reference server code Enhancement New feature or request Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod Redis Issues related to Redis Up for grabs Issues that have little to no dependencies on design/product, can be picked up in an ad-hoc manner

Comments

@trishaanand
Copy link
Contributor

trishaanand commented Jul 27, 2021 •
edited
Loading

Summary

When response type of List is returned, the plugin throws an unable to case exception

Motivation

Commands like scan 0 on redis returns a response of List of List which is not supported. keys * (which returns response in a supported format )is a heavy op and does not work well in case of clusters.
@trishaanand trishaanand added Enhancement New feature or request Core Query Execution Issues related to the execution of all queries labels Jul 27, 2021
@sumitsum sumitsum mentioned this issue Aug 23, 2021
Closed
@ajinkyakulkarni ajinkyakulkarni removed this from the AP-RK Issues - Sep 21 milestone Aug 23, 2021
@Nikhil-Nandagopal Nikhil-Nandagopal added the Redis Issues related to Redis label Oct 6, 2021
@github-actions github-actions bot added the Query & JS Pod Issues related to the query & JS Pod label Oct 6, 2021
@rohan-arthur
Copy link
Contributor

rohan-arthur commented Apr 7, 2022 •
edited by Nikhil-Nandagopal
Loading

Stats

Stat Values
Reach 42
Effort (months) 0.5

@Nikhil-Nandagopal Nikhil-Nandagopal changed the title [Feature][Redis Plugin] Add support for response type of List<List> [Feature]-[42]: Apr 8, 2022
@rohan-arthur rohan-arthur changed the title [Feature]-[42]: [Feature]-[42]:[Redis Plugin] Add support for response type of List<List> Apr 10, 2022
@Nikhil-Nandagopal Nikhil-Nandagopal added Integrations Product Issues related to a specific integration and removed Core Query Execution Issues related to the execution of all queries labels Sep 12, 2022
@sumitsum sumitsum added Up for grabs Issues that have little to no dependencies on design/product, can be picked up in an ad-hoc manner Backend This marks the issue or pull request to reference server code labels Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Backend This marks the issue or pull request to reference server code Enhancement New feature or request Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod Redis Issues related to Redis Up for grabs Issues that have little to no dependencies on design/product, can be picked up in an ad-hoc manner
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@ajinkyakulkarni @sumitsum @Nikhil-Nandagopal @trishaanand @areyabhishek @appsmith-bot @rohan-arthur