> On March 16, 2015, 9:54 p.m., Aman Sinha wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project/ProjectRecordBatch.java, > > line 369 > > <https://reviews.apache.org/r/32089/diff/2/?file=896259#file896259line369> > > > > Can you add some comments here to indicate which columns would be > > considered here and about uniquely naming the columns.
Updated!!! - Sean Hsuan-Yi ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32089/#review76653 ----------------------------------------------------------- On March 16, 2015, 7:45 p.m., Sean Hsuan-Yi Chu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32089/ > ----------------------------------------------------------- > > (Updated March 16, 2015, 7:45 p.m.) > > > Review request for drill and Aman Sinha. > > > Bugs: DRILL-2311 > https://issues.apache.org/jira/browse/DRILL-2311 > > > Repository: drill-git > > > Description > ------- > > In ProjectRecordBatch, ensure the output columns are unique. > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/project/ProjectRecordBatch.java > 8f7812f > exec/java-exec/src/test/java/org/apache/drill/TestExampleQueries.java > 8ae0ae4 > > Diff: https://reviews.apache.org/r/32089/diff/ > > > Testing > ------- > > Unit tests, all QA tests > > > Thanks, > > Sean Hsuan-Yi Chu > >
