diff options
author | Hanifi Gunes <hgunes@maprtech.com> | 2014-10-21 23:16:47 -0700 |
---|---|---|
committer | Jinfeng Ni <jni@maprtech.com> | 2014-11-07 10:50:56 -0800 |
commit | 76df2ffe6a7ad999557fdf5f1bdf3d76288e65a3 (patch) | |
tree | da50fd91622e28e56aa3c1aceccbd70ceda385eb | |
parent | 70993d2d44a02c8edc48a7dc2ea0b79bded82d28 (diff) |
DRILL-1539: fix an issue regarding populateEmpties in RepeatedMapVector
-rw-r--r-- | exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java b/exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java index d73aa7f54..99b94530f 100644 --- a/exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java +++ b/exec/java-exec/src/main/java/org/apache/drill/exec/vector/complex/RepeatedMapVector.java @@ -360,6 +360,7 @@ public class RepeatedMapVector extends AbstractContainerVector implements Repeat public boolean copyValueSafe(int srcIndex, int destIndex) { RepeatedMapHolder holder = new RepeatedMapHolder(); accessor.get(srcIndex, holder); + to.populateEmpties(destIndex+1); int newIndex = to.offsets.getAccessor().get(destIndex); //todo: make these bulk copies for (int i = holder.start; i < holder.end; i++, newIndex++) { @@ -372,7 +373,6 @@ public class RepeatedMapVector extends AbstractContainerVector implements Repeat if (!to.offsets.getMutator().setSafe(destIndex+1, newIndex)) { return false; } - to.lastPopulatedValueIndex = destIndex; return true; } @@ -609,6 +609,7 @@ public class RepeatedMapVector extends AbstractContainerVector implements Repeat @Override public void clear() { getMutator().reset(); + offsets.clear(); for(ValueVector v : vectors.values()) { v.clear();; |