Re: Review Request 36486: HIVE-11262 Skip MapJoin processing if the join hash table is empty

2015-07-16 Thread Vikram Dixit Kumaraswamy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36486/#review91926 --- Ship it! Ship It! - Vikram Dixit Kumaraswamy On July 16, 2015,

Re: Review Request 36486: HIVE-11262 Skip MapJoin processing if the join hash table is empty

2015-07-15 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36486/ --- (Updated July 16, 2015, 12:45 a.m.) Review request for hive, Matt McCline,

Review Request 36486: HIVE-11262 Skip MapJoin processing if the join hash table is empty

2015-07-14 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36486/ --- Review request for hive, Matt McCline, Vikram Dixit Kumaraswamy, and Wei Zheng.

Re: Review Request 36486: HIVE-11262 Skip MapJoin processing if the join hash table is empty

2015-07-14 Thread Matt McCline
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36486/#review91674 --- ql/src/java/org/apache/hadoop/hive/ql/exec/MapJoinOperator.java

Re: Review Request 36486: HIVE-11262 Skip MapJoin processing if the join hash table is empty

2015-07-14 Thread Jason Dere
On July 14, 2015, 11:06 p.m., Matt McCline wrote: ql/src/java/org/apache/hadoop/hive/ql/exec/MapJoinOperator.java, line 667 https://reviews.apache.org/r/36486/diff/1/?file=1011846#file1011846line667 Why isn't the condition this.hashTblInitedOnce included anymore?