Record that the result of DictionaryBlock#copyPositions is compact

prestodb · Mar 9, 2022 · 0280165 · 0280165
1 parent 4871cee
commit 0280165
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/presto-common/src/main/java/com/facebook/presto/common/block/DictionaryBlock.java b/presto-common/src/main/java/com/facebook/presto/common/block/DictionaryBlock.java
@@ -408,7 +408,11 @@ public Block copyPositions(int[] positions, int offset, int length)
             }
             newIds[i] = oldIndexToNewIndex.get(oldIndex);
         }
-        return new DictionaryBlock(dictionary.copyPositions(positionsToCopy.elements(), 0, positionsToCopy.size()), newIds);
+        return new DictionaryBlock(
+                length,
+                dictionary.copyPositions(positionsToCopy.elements(), 0, positionsToCopy.size()),
+                newIds,
+                true); // new dictionary is compact
     }
 
     @Override