You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by Steven Phillips <sp...@maprtech.com> on 2014/05/29 07:33:05 UTC

Review Request 22011: DRILL-854: PartitionSender generated code is too large

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

Review request for drill.


Repository: drill-git


Description
-------

This patch restructures the PartitionSender, changing OutgoingRecordBatch to an inner class of the generated code. This way we don't have a reference for every single outgoing vector in the generated code.


Diffs
-----

  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/OutgoingRecordBatch.java c86da8c 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java d0eaf9a 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/Partitioner.java 7d3998b 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionerTemplate.java fe62b73 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/StatusHandler.java PRE-CREATION 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/svremover/RemovingRecordBatch.java 41b71d0 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/xsort/ExternalSortBatch.java 2289680 
  exec/java-exec/src/main/java/org/apache/drill/exec/vector/CopyUtil.java PRE-CREATION 

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


Testing
-------


Thanks,

Steven Phillips