[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16074278#comment-16074278 ] Hive QA commented on HIVE-16659: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16074277#comment-16074277 ] Rui Li commented on HIVE-16659: --- +1 pending test > Query plan should reflect hive.spark.use.groupby.shuffle

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16073370#comment-16073370 ] Rui Li commented on HIVE-16659: --- Thanks Bing for the update. I left some minor comments on RB. > Query plan

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16073343#comment-16073343 ] Bing Li commented on HIVE-16659: [~ruili], I updated the patch based on your comment and add the link of

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16073292#comment-16073292 ] Hive QA commented on HIVE-16659: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16071530#comment-16071530 ] Hive QA commented on HIVE-16659: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16071514#comment-16071514 ] Bing Li commented on HIVE-16659: Hi, [~ruili] Thank you for the review! I checked the latest code on

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16070297#comment-16070297 ] Rui Li commented on HIVE-16659: --- Thanks [~libing] for working on this. Patch looks good overall. Can you

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16043076#comment-16043076 ] Chao Sun commented on HIVE-16659: - Agree. Maybe we can have some new edge properties to reflect this, and

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16042389#comment-16042389 ] Rui Li commented on HIVE-16659: --- [~libing] sure, thanks for helping. Some suggestions: instead of enforcing

[jira] [Commented] (HIVE-16659) Query plan should reflect hive.spark.use.groupby.shuffle

[ https://issues.apache.org/jira/browse/HIVE-16659?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16042361#comment-16042361 ] Bing Li commented on HIVE-16659: Hi, [~ruili] Could I take it over? > Query plan should reflect