You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@drill.apache.org by pa...@apache.org on 2018/06/02 04:50:11 UTC

[drill] 03/10: DRILL-6343: bit vector copyFromSafe is not doing realloc

This is an automated email from the ASF dual-hosted git repository.

parthc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git

commit 5ef220e70fb898500110eabdbbe1b0bf6401049b
Author: Padma Penumarthy <pp...@yahoo.com>
AuthorDate: Thu Apr 19 14:36:38 2018 -0700

    DRILL-6343: bit vector copyFromSafe is not doing realloc
---
 .../src/main/java/org/apache/drill/exec/vector/BitVector.java     | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/exec/vector/src/main/java/org/apache/drill/exec/vector/BitVector.java b/exec/vector/src/main/java/org/apache/drill/exec/vector/BitVector.java
index 2473556..3725364 100644
--- a/exec/vector/src/main/java/org/apache/drill/exec/vector/BitVector.java
+++ b/exec/vector/src/main/java/org/apache/drill/exec/vector/BitVector.java
@@ -220,13 +220,11 @@ public final class BitVector extends BaseDataValueVector implements FixedWidthVe
     this.mutator.set(outIndex, from.accessor.get(inIndex));
   }
 
-  public boolean copyFromSafe(int inIndex, int outIndex, BitVector from) {
-    if (outIndex >= this.getValueCapacity()) {
-      decrementAllocationMonitor();
-      return false;
+  public void copyFromSafe(int inIndex, int outIndex, BitVector from) {
+    while (outIndex >= this.getValueCapacity()) {
+      reAlloc();
     }
     copyFrom(inIndex, outIndex, from);
-    return true;
   }
 
   @Override

-- 
To stop receiving notification emails like this one, please contact
parthc@apache.org.