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

[YCQL] Check support for OFFSET with aggregates (like MIN) #15086

Open
pkj415 opened this issue Nov 21, 2022 · 1 comment
Open

[YCQL] Check support for OFFSET with aggregates (like MIN) #15086

pkj415 opened this issue Nov 21, 2022 · 1 comment
Labels
area/ycql Yugabyte CQL (YCQL) kind/bug This issue is a bug priority/low Low priority
Projects

Comments

@pkj415
Copy link
Contributor

pkj415 commented Nov 21, 2022

Jira Link: DB-4302

Description

We allow OFFSET in SELECT queries in YCQL, but apache cassandra doesn't (https://cassandra.apache.org/doc/latest/cassandra/cql/dml.html)

Given this, did we intend to support aggregates queries like MIN with OFFSET?

If we did, check if it works correctly and add tests. If not, ensure that it errors out explicitly.

@pkj415 pkj415 added area/ycql Yugabyte CQL (YCQL) status/awaiting-triage Issue awaiting triage labels Nov 21, 2022
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Nov 21, 2022
@OlegLoginov
Copy link
Contributor

OlegLoginov commented Jan 19, 2023

OFFSET can be useful for an aggregate function like AVG or MIN if you want to get a result for a random subset of rows.
E.g. for 1 or for a few tablets.
That's why I've not disabled OFFSET for aggregate functions.
(In the fix for #13295.)

@OlegLoginov OlegLoginov added this to To Do in YCQL via automation Jan 19, 2023
@yugabyte-ci yugabyte-ci removed the status/awaiting-triage Issue awaiting triage label Jan 23, 2023
@OlegLoginov OlegLoginov added priority/low Low priority and removed priority/medium Medium priority issue labels Jan 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ycql Yugabyte CQL (YCQL) kind/bug This issue is a bug priority/low Low priority
Projects
YCQL
  
To Do
Development

No branches or pull requests

3 participants