[jira] [Commented] (HIVE-12180) Use MapJoinDesc::isHybridHashJoin() instead of the HiveConf lookup in Vectorizer

2015-10-15 Thread Hive QA (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-12180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14958929#comment-14958929 ] Hive QA commented on HIVE-12180: Here are the results of testing the latest attachment:

[jira] [Commented] (HIVE-12180) Use MapJoinDesc::isHybridHashJoin() instead of the HiveConf lookup in Vectorizer

2015-10-15 Thread Matt McCline (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-12180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14959776#comment-14959776 ] Matt McCline commented on HIVE-12180: - Committed to master. > Use MapJoinDesc::isHybridHashJoin()

[jira] [Commented] (HIVE-12180) Use MapJoinDesc::isHybridHashJoin() instead of the HiveConf lookup in Vectorizer

2015-10-14 Thread Wei Zheng (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-12180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14957873#comment-14957873 ] Wei Zheng commented on HIVE-12180: -- The reason for doing this is that there are cases Hybrid is on in

[jira] [Commented] (HIVE-12180) Use MapJoinDesc::isHybridHashJoin() instead of the HiveConf lookup in Vectorizer

2015-10-14 Thread Wei Zheng (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-12180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14957874#comment-14957874 ] Wei Zheng commented on HIVE-12180: -- [~mmccline] Can you take a look? It's one line change. > Use

[jira] [Commented] (HIVE-12180) Use MapJoinDesc::isHybridHashJoin() instead of the HiveConf lookup in Vectorizer

2015-10-14 Thread Matt McCline (JIRA)
[ https://issues.apache.org/jira/browse/HIVE-12180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=14957881#comment-14957881 ] Matt McCline commented on HIVE-12180: - +1 LGTM > Use MapJoinDesc::isHybridHashJoin() instead of the