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

Maybe we need the new query result builder for redis commands. #35

Open
fastio opened this issue Mar 10, 2019 · 1 comment
Open

Maybe we need the new query result builder for redis commands. #35

fastio opened this issue Mar 10, 2019 · 1 comment

Comments

@fastio
Copy link
Owner

fastio commented Mar 10, 2019

  1. We should build the redis result directly, instead of building over the `query_result_builder.
  2. For the lists commands, such as `lrange, we do not read all the clustering row of the paritition.
@fastio
Copy link
Owner Author

fastio commented Mar 10, 2019

The redis result builder will consume the data provided by flat reader. After the fetching data proccess was compeleted, flush it to the connection directly.

@fastio fastio changed the title May be we nneed the new query result builder for redis commands. May be we need the new query result builder for redis commands. Mar 10, 2019
@fastio fastio changed the title May be we need the new query result builder for redis commands. Maybe we need the new query result builder for redis commands. Mar 11, 2019
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