Use primitive null packing in RowBlockEncoding

pettyjamesm · pettyjamesm · commit fd6b1bb39f05 · 2025-08-18T16:21:38.000-04:00
diff --git a/core/trino-spi/src/main/java/io/trino/spi/block/RowBlock.java b/core/trino-spi/src/main/java/io/trino/spi/block/RowBlock.java
@@ -148,6 +148,7 @@ public boolean hasNull()
         return false;
     }
 
+    @Nullable
     boolean[] getRawRowIsNull()
     {
         return rowIsNull;
diff --git a/core/trino-spi/src/main/java/io/trino/spi/block/RowBlockEncoding.java b/core/trino-spi/src/main/java/io/trino/spi/block/RowBlockEncoding.java
@@ -49,7 +49,7 @@ public void writeBlock(BlockEncodingSerde blockEncodingSerde, SliceOutput sliceO
             blockEncodingSerde.writeBlock(sliceOutput, rawFieldBlock);
         }
 
-        EncoderUtil.encodeNullsAsBits(sliceOutput, block);
+        EncoderUtil.encodeNullsAsBits(sliceOutput, rowBlock.getRawRowIsNull(), 0, rowBlock.getPositionCount());
     }
 
     @Override

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public void writeBlock(BlockEncodingSerde blockEncodingSerde, SliceOutput sliceO`
`49`	`49`	`blockEncodingSerde.writeBlock(sliceOutput, rawFieldBlock);`
`50`	`50`	`}`
`51`	`51`
`52`		`- EncoderUtil.encodeNullsAsBits(sliceOutput, block);`
	`52`	`+ EncoderUtil.encodeNullsAsBits(sliceOutput, rowBlock.getRawRowIsNull(), 0, rowBlock.getPositionCount());`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`@Override`