[ 
https://issues.apache.org/jira/browse/CASSANDRA-16427?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17282067#comment-17282067
 ] 

Caleb Rackliffe edited comment on CASSANDRA-16427 at 2/9/21, 10:21 PM:
-----------------------------------------------------------------------

LGTM, just a couple minor things...
 - -Should the new {{GroupByTest}} exist in the 3.0 patch?-
 - On the trunk patch, {{GroupByTest}}, line 61, there's a dangling closing 
parenthesis.
 - {{public static Iterator<Object[]> toObjects(ResultSet rs)}} looks like it 
was moved in the 3.0 branch but not trunk.


was (Author: maedhroz):
LGTM, just a couple minor things...

- Should the new {{GroupByTest}} exist in the 3.0 patch?
- On the trunk patch, {{GroupByTest}}, line 61, there's a dangling closing 
parenthesis.
- {{public static Iterator<Object[]> toObjects(ResultSet rs)}} looks like it 
was moved in the 3.0 branch but not trunk.

> In-JVM dtest paging does not handle Group By correctly
> ------------------------------------------------------
>
>                 Key: CASSANDRA-16427
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-16427
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Test/dtest/java
>            Reporter: Alex Petrov
>            Assignee: Alex Petrov
>            Priority: Normal
>
> In-JVM dtest paging is using a pager that disregards the type of the executed 
> query, resulting into `GROUP BY` queries being executed like normal SELECT 
> queries without GROUP BY clause.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to