---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12767/
---
Review request for hive.
Bugs: HIVE-4877
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12767/#review23531
---
ql/src/java/org/apache/hadoop/hive/ql/exec/MuxOperator.java
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12767/#review23541
---
Ship it!
Ship It!
- Ashutosh Chauhan
On July 19, 2013, 7:04
On July 19, 2013, 6:02 p.m., Ashutosh Chauhan wrote:
ql/src/java/org/apache/hadoop/hive/ql/exec/DemuxOperator.java, lines 229-231
https://reviews.apache.org/r/12767/diff/1/?file=323660#file323660line229
I didn't get why there is an if check here? Can you add a comment
explaining
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12767/
---
(Updated July 19, 2013, 7:04 p.m.)
Review request for hive.
Changes
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12767/
---
(Updated July 19, 2013, 8:48 p.m.)
Review request for hive.
Changes
---
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12767/#review23527
---
Good work, Yin! some minor comments.
On July 19, 2013, 6:25 p.m., Ashutosh Chauhan wrote:
ql/src/java/org/apache/hadoop/hive/ql/exec/MuxOperator.java, line 299
https://reviews.apache.org/r/12767/diff/1/?file=323661#file323661line299
Should we also add following in comment?
.. and directly call process on children