drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Hsuan-Yi Chu" <hsua...@usc.edu>
Subject Re: Review Request 32089: DRILL-2311: Create table with same columns of different case results in a java.lang.IllegalStateException
Date Mon, 16 Mar 2015 21:09:20 GMT


> On March 16, 2015, 7:58 p.m., Jinfeng Ni wrote:
> > exec/java-exec/src/test/java/org/apache/drill/TestExampleQueries.java, line 44
> > <https://reviews.apache.org/r/32089/diff/2/?file=896260#file896260line44>
> >
> >     I understand these testcase should use "unOrdered()", in stead of "ordered().
But why do you change in this patch? Is it because you run into testcase failure with the
original "ordered()", once you change ProjectRecordBatch? If so, then, we need understand
why the testcase previously is successful with "ordered", and now fails with "ordered" with
your code change in ProjectRecordBatch.

Oh...oh... 
Actually, either worked just fine. I made the change simply because it is not theoretically
correct.


- Sean Hsuan-Yi


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32089/#review76631
-----------------------------------------------------------


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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message