Re: Review Request 30602: HIVE-6099

2015-02-05 Thread Alexander Pivovarov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30602/#review71248 --- ql/src/java/org/apache/hadoop/hive/ql/plan/GroupByDesc.java

Re: Review Request 30602: HIVE-6099

2015-02-04 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30602/ --- (Updated Feb. 5, 2015, 6:06 a.m.) Review request for hive and Navis Ryu. Chan

Re: Review Request 30602: HIVE-6099

2015-02-04 Thread Navis Ryu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30602/#review71156 --- ql/src/java/org/apache/hadoop/hive/ql/exec/GroupByOperator.java

Re: Review Request 30602: HIVE-6099

2015-02-04 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30602/ --- (Updated Feb. 4, 2015, 9:16 p.m.) Review request for hive and Navis Ryu. Chan

Re: Review Request 30602: HIVE-6099

2015-02-03 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30602/ --- (Updated Feb. 4, 2015, 2:47 a.m.) Review request for hive and Navis Ryu. Chan

Re: Review Request 30602: HIVE-6099

2015-02-03 Thread Navis Ryu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30602/#review70902 --- We can simplify code further genGroupByPlanGroupByOperator1.distPar