---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28889/#review64959
---
Patch looks good. One suggestion, we should be able to change the
On Dec. 12, 2014, 7:45 p.m., Xuefu Zhang wrote:
ql/src/java/org/apache/hadoop/hive/ql/optimizer/SparkMapJoinProcessor.java,
line 78
https://reviews.apache.org/r/28889/diff/2/?file=789801#file789801line78
nit: grandParentOps.get(0) is repeated in the next line. nice to have a
On Dec. 12, 2014, 7:45 p.m., Xuefu Zhang wrote:
Patch looks good. One suggestion, we should be able to change the static
methods non-static, which would further simplify the code.
I agree. Let me change it.
- Chao
---
This is an
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28889/
---
(Updated Dec. 13, 2014, 12:24 a.m.)
Review request for hive, Szehon Ho and
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28889/
---
(Updated Dec. 11, 2014, 10:36 p.m.)
Review request for hive, Szehon Ho and
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28889/
---
Review request for hive, Szehon Ho and Xuefu Zhang.
Bugs: HIVE-8911