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

(Updated Aug. 20, 2013, 12:10 p.m.)


Review request for drill.


Changes
-------

Sorry for the late reply, I'd been pretty occupied for the last days.

I guess AbstractSingleRecordBatch is for single input? so I just go from 
AbstractRecordBatch?
 


Repository: drill-git


Description
-------

union pop.


Diffs (updated)
-----

  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractMultiple.java
 PRE-CREATION 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractPhysicalVisitor.java
 3b58803 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/PhysicalVisitor.java
 8e09e3a 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/Union.java
 PRE-CREATION 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ImplCreator.java
 2ce5c28 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScreenCreator.java
 9b31407 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionBatchCreator.java
 PRE-CREATION 
  
sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/union/UnionRecordBatch.java
 PRE-CREATION 
  
sandbox/prototype/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/TestSimpleFragmentRun.java
 6587237 
  
sandbox/prototype/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/filter/TestSimpleFilter.java
 afb0b3a 
  
sandbox/prototype/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/union/TestSimpleUnion.java
 PRE-CREATION 
  sandbox/prototype/exec/java-exec/src/test/resources/filter/test1.json 4f82145 
  sandbox/prototype/exec/java-exec/src/test/resources/union/test1.json 
PRE-CREATION 

Diff: https://reviews.apache.org/r/12960/diff/


Testing
-------

org.apache.drill.exec.physical.impl.union.TestSimpleUnion


File Attachments
----------------


  
https://reviews.apache.org/media/uploaded/files/2013/07/30/DRILL-156.0002.patch.txt


Thanks,

Lisen Mu

Reply via email to