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

OFFSET + LIMIT queries cause cqlsh to go into pagination mode #415

Closed
robertpang opened this issue Aug 1, 2018 · 1 comment
Closed

OFFSET + LIMIT queries cause cqlsh to go into pagination mode #415

robertpang opened this issue Aug 1, 2018 · 1 comment
Assignees
@robertpang
Labels
kind/bug This issue is a bug

Comments

@robertpang
Copy link
Contributor

The following query doesn't seem to stop even when it already has the needed number of rows.

cqlsh:ybdemo_keyspace> select * from cassandrakeyvalue offset 50 limit 10;

 k          | v                      | val  | description
------------+------------------------+------+-------------
 key:996715 | 0x76616c3a393936373135 | null |        null
 key:997845 | 0x76616c3a393937383435 | null |        null
 key:136114 | 0x76616c3a313336313134 | null |        null
 key:349995 | 0x76616c3a333439393935 | null |        null
 key:387948 | 0x76616c3a333837393438 | null |        null
 key:536506 | 0x76616c3a353336353036 | null |        null
 key:587682 | 0x76616c3a353837363832 | null |        null
 key:597575 | 0x76616c3a353937353735 | null |        null
 key:605191 | 0x76616c3a363035313931 | null |        null
 key:619590 | 0x76616c3a363139353930 | null |        null

---MORE---
---MORE---
---MORE---
---MORE---
....

Doesn't seem to return even after lots of ENTER keystrokes.
@robertpang robertpang self-assigned this Aug 1, 2018
@robertpang robertpang added the kind/bug This issue is a bug label Aug 1, 2018
yugabyte-ci pushed a commit that referenced this issue Aug 2, 2018
@robertpang
#415: End a query always when LIMIT clause is hit.
8765bab 
Summary: Fix an issue OFFSET + LIMIT queries cause cqlsh to go into pagination mode. YCQL service should return an empty paging state to indicate the query has completed.

Test Plan: TestSelect.testSelectWithOffset verifying paging state is empty after query completes.

Reviewers: mihnea

Reviewed By: mihnea

Subscribers: yql

Differential Revision: https://phabricator.dev.yugabyte.com/D5250
@robertpang
Copy link
Contributor Author

Fixed in commit 8765bab.

@ryan-ally ryan-ally mentioned this issue Dec 21, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@robertpang robertpang

Labels
kind/bug This issue is a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@robertpang