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

ASF GitHub Bot commented on PHOENIX-2257:
-----------------------------------------

Github user JamesRTaylor commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/120#discussion_r41285998
  
    --- Diff: 
phoenix-core/src/it/java/org/apache/phoenix/end2end/DerivedTableIT.java ---
    @@ -313,21 +290,25 @@ public void testDerivedTableWithGroupBy() throws 
Exception {
                 query = "SELECT DISTINCT COLLECTDISTINCT(t.b) FROM (SELECT 
b_string b, a_string a FROM aTable GROUP BY a_string, b_string) AS t GROUP BY 
t.a";
    --- End diff --
    
    @maryannxue - do you think the underlying order returned is indeterminate? 
For our tests, we have phoenix.query.force.rowkeyorder set to true, so 
underlying scans will return in row key order. Just want to make sure 
@chiastic-security fix isn't masking a bug.


> Failing integration test in DerivedTableIT
> ------------------------------------------
>
>                 Key: PHOENIX-2257
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2257
>             Project: Phoenix
>          Issue Type: Bug
>    Affects Versions: 4.6.0
>         Environment: Ubuntu and Fedora, 64-bit Java 8
>            Reporter: James Heather
>         Attachments: org.apache.phoenix.end2end.DerivedTableIT-output.txt
>
>
> {code}
> Failed tests:
>   DerivedTableIT.testDerivedTableWithGroupBy:320 expected:<['e']> but 
> was:<['b', 'c', 'e']>
>   DerivedTableIT.testDerivedTableWithGroupBy:320 expected:<['e']> but 
> was:<['b', 'c', 'e']>
> {code}
> This is on a freshly cloned copy of the repo, on the master branch.
> In the output, I get a couple of {{TableInfoMissingExceptions}}, which might 
> be related.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to