drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Venki Korukanti" <venki.koruka...@gmail.com>
Subject Re: Review Request 30910: DRILL-2178: Update outgoing record batch size and allocation in PartitionSender
Date Thu, 12 Feb 2015 17:21:21 GMT

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

(Updated Feb. 12, 2015, 5:21 p.m.)


Review request for drill and Steven Phillips.


Changes
-------

Add a method to set initial capacity of ValueVectors. Initial memory allocated using allocateNew()
is sufficient to fit the given initial capacity.


Repository: drill-git


Description
-------

Please see the JIRA DRILL-2178 for details


Diffs (updated)
-----

  exec/java-exec/src/main/codegen/templates/FixedValueVectors.java 52a3868 
  exec/java-exec/src/main/codegen/templates/NullableValueVectors.java ba7c629 
  exec/java-exec/src/main/codegen/templates/RepeatedValueVectors.java d39040e 
  exec/java-exec/src/main/codegen/templates/VariableLengthVectors.java aa5b702 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/materialize/QueryWritableBatch.java
cef4101 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java
f09acaa 
  exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionerTemplate.java
4292c09 
  exec/java-exec/src/main/java/org/apache/drill/exec/record/FragmentWritableBatch.java d122311

  exec/java-exec/src/main/java/org/apache/drill/exec/record/MaterializedField.java bcc226f

  exec/java-exec/src/main/java/org/apache/drill/exec/vector/BitVector.java f6644bd 
  exec/java-exec/src/main/java/org/apache/drill/exec/vector/ObjectVector.java 3c15db3 
  exec/java-exec/src/main/java/org/apache/drill/exec/vector/RepeatedVector.java b23ee02 
  exec/java-exec/src/main/java/org/apache/drill/exec/vector/ValueVector.java df6a486 
  exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/MapVector.java c5dc5ba

  exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedListVector.java
3078f4e 
  exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java
6dce363 
  exec/java-exec/src/test/java/org/apache/drill/exec/record/vector/TestValueVector.java 2bed433


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


Testing
-------


Thanks,

Venki Korukanti


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