[jira] [Commented] (PHOENIX-2820) Investigate why SortMergeJoinIT has a sort in the explain plan
[ https://issues.apache.org/jira/browse/PHOENIX-2820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15231706#comment-15231706 ] Hudson commented on PHOENIX-2820: - FAILURE: Integrated in Phoenix-master #1186 (See [https://builds.apache.org/job/Phoenix-master/1186/]) PHOENIX-2820 Investigate why SortMergeJoinIT has a sort in the explain (jtaylor: rev 91800b703d9879574ac5b3ee6fcfe6dd73af2148) * phoenix-core/src/it/java/org/apache/phoenix/end2end/SubqueryUsingSortMergeJoinIT.java * phoenix-core/src/main/java/org/apache/phoenix/compile/GroupByCompiler.java * phoenix-core/src/it/java/org/apache/phoenix/end2end/SortMergeJoinIT.java > Investigate why SortMergeJoinIT has a sort in the explain plan > --- > > Key: PHOENIX-2820 > URL: https://issues.apache.org/jira/browse/PHOENIX-2820 > Project: Phoenix > Issue Type: Bug >Reporter: James Taylor >Assignee: James Taylor > Fix For: 4.8.0 > > Attachments: PHOENIX-2820.patch > > > After removing an unnecessary merge sort for aggregate queries in this[1] > commit, a sort started appearing in one of the explain plans for > SortMergeJoinIT. This sort is not required, so we need to figure out why it > is part of the explain plan now. > [1]https://github.com/apache/phoenix/commit/15766625ab9a132f8d8ac625b026a5c56e62e879, > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (PHOENIX-2820) Investigate why SortMergeJoinIT has a sort in the explain plan
[ https://issues.apache.org/jira/browse/PHOENIX-2820?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15230378#comment-15230378 ] Maryann Xue commented on PHOENIX-2820: -- [~jamestaylor], just checked, it was PHOENIX-2758 that caused this regression, instead of PHOENIX-2802. > Investigate why SortMergeJoinIT has a sort in the explain plan > --- > > Key: PHOENIX-2820 > URL: https://issues.apache.org/jira/browse/PHOENIX-2820 > Project: Phoenix > Issue Type: Bug >Reporter: James Taylor > Fix For: 4.8.0 > > > After removing an unnecessary merge sort for aggregate queries in this[1] > commit, a sort started appearing in one of the explain plans for > SortMergeJoinIT. This sort is not required, so we need to figure out why it > is part of the explain plan now. > [1]https://github.com/apache/phoenix/commit/15766625ab9a132f8d8ac625b026a5c56e62e879, > -- This message was sent by Atlassian JIRA (v6.3.4#6332)
