You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by Sean Hsuan-Yi Chu <hs...@usc.edu> on 2015/04/01 19:43:43 UTC
Re: Review Request 32590: DRILL-2591: In UnionAllRecordBactch,
the mechansim to detect schema change is corrected
> On March 29, 2015, 5:28 p.m., Aman Sinha wrote:
> > exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionAllRecordBatch.java, line 357
> > <https://reviews.apache.org/r/32590/diff/1/?file=908305#file908305line357>
> >
> > The error message says 'right input' - it should be 'left input'
Updated!
> On March 29, 2015, 5:28 p.m., Aman Sinha wrote:
> > exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionAllRecordBatch.java, line 356
> > <https://reviews.apache.org/r/32590/diff/1/?file=908305#file908305line356>
> >
> > Same as above.
Same as above.
> On March 29, 2015, 5:28 p.m., Aman Sinha wrote:
> > exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionAllRecordBatch.java, line 330
> > <https://reviews.apache.org/r/32590/diff/1/?file=908305#file908305line330>
> >
> > This does not look right...the check here is checking schema change status of the outgoing batch of UnionAll, wherease you are interested in the incoming batch.
A new approach is to memorize the schema for the first record batch (from the left and right sides). And just the schemas for the subsequent record batches with the memorized schema. If they are different, that means schema has been changed.
- Sean Hsuan-Yi
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32590/#review78163
-----------------------------------------------------------
On March 27, 2015, 8:33 p.m., Sean Hsuan-Yi Chu wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/32590/
> -----------------------------------------------------------
>
> (Updated March 27, 2015, 8:33 p.m.)
>
>
> Review request for drill and Aman Sinha.
>
>
> Bugs: DRILL-2591.1
> https://issues.apache.org/jira/browse/DRILL-2591.1
>
>
> Repository: drill-git
>
>
> Description
> -------
>
> DRILL-2591: In UnionAllRecordBactch, the mechansim to detect schema change is corrected
>
>
> Diffs
> -----
>
> exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionAllRecordBatch.java 806104a
> exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/visitor/FinalColumnReorderer.java 1aa033b
> exec/java-exec/src/test/java/org/apache/drill/TestUnionAll.java 520c204
> exec/java-exec/src/test/resources/store/json/dateData.json PRE-CREATION
> exec/java-exec/src/test/resources/store/json/timeStmpData.json PRE-CREATION
> exec/java-exec/src/test/resources/testframework/testUnionAllQueries/q18.tsv PRE-CREATION
>
> Diff: https://reviews.apache.org/r/32590/diff/
>
>
> Testing
> -------
>
> QA, unit all passed
>
>
> Thanks,
>
> Sean Hsuan-Yi Chu
>
>