Re: Review Request 26725: HIVE-8462 CBO duplicates column

2014-10-15 Thread John Pullokkaran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26725/#review56841 --- Ship it! Ship It! - John Pullokkaran On Oct. 15, 2014, 11:07 p.m

Re: Review Request 26725: HIVE-8462 CBO duplicates column

2014-10-15 Thread Sergey Shelukhin
> On Oct. 15, 2014, 9:30 p.m., John Pullokkaran wrote: > > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java, line 2710 > > > > > > Since this fix relies on internal name, i am not sure if it will work

Re: Review Request 26725: HIVE-8462 CBO duplicates column

2014-10-15 Thread Sergey Shelukhin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26725/ --- (Updated Oct. 15, 2014, 11:07 p.m.) Review request for hive, Ashutosh Chauhan a

Re: Review Request 26725: HIVE-8462 CBO duplicates column

2014-10-15 Thread John Pullokkaran
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26725/#review56816 --- ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java <

Review Request 26725: HIVE-8462 CBO duplicates column

2014-10-14 Thread Sergey Shelukhin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26725/ --- Review request for hive, Ashutosh Chauhan and John Pullokkaran. Repository: hiv