refactor(reader): FlatSegmentDecoder is SerializedArrayDecoder

dfa1 · claude · dfa1 · commit 3a657a320d8a · 2026-07-04T13:48:55.000+02:00
Renamed after its exact Rust counterpart (SerializedArray in
vortex-array/src/serde.rs: "a parsed but not-yet-decoded
deserialized array" whose decode() resolves the encoding id against
the spec table and consults the registry). "Flat" is a layout
concept and "segment" a byte-range concept — the unit this class
decodes is one serialized array message. VortexHandle's
decodeFlatSegment follows as decodeSegment, next to rawSegment.
Pitest target FQN and living docs updated; released changelog
entries and ADR 0001 stay as written.

Co-Authored-By: Claude Fable 5 &lt;noreply@anthropic.com&gt;
diff --git a/cli/src/main/java/io/github/dfa1/vortex/cli/tui/VortexInspectorTui.java b/cli/src/main/java/io/github/dfa1/vortex/cli/tui/VortexInspectorTui.java
@@ -607,7 +607,7 @@ private void runDictLoad(InspectorTree.Node dictNode) {
                     int segIdx = values.segments().getFirst();
                     SegmentSpec spec = tree.segmentSpecs().get(segIdx);
                     io.github.dfa1.vortex.reader.array.Array arr =
-                            handle.decodeFlatSegment(spec, dtype, values.rowCount(), arena);
+                            handle.decodeSegment(spec, dtype, values.rowCount(), arena);
                     int n = (int) Math.min(arr.length(), DATA_PREVIEW_ROWS);
                     List<String> out = new ArrayList<>(n);
                     for (int i = 0; i < n; i++) {
@@ -722,7 +722,7 @@ private List<String> decodeStatsFlat(
             }
             int segIdx = flat.segments().getFirst();
             SegmentSpec spec = tree.segmentSpecs().get(segIdx);
-            Array arr = handle.decodeFlatSegment(spec, statsDtype, flat.rowCount(), arena);
+            Array arr = handle.decodeSegment(spec, statsDtype, flat.rowCount(), arena);
             return InspectorRender.formatStatsArray(arr, statsDtype);
         }
 
diff --git a/docs/testing.md b/docs/testing.md
@@ -113,7 +113,7 @@ bounds/parse classes** — not the whole codebase. It measures whether the tests
 ```bash
 ./mvnw -pl core   -P pitest verify              # IoBounds, PTypeIO
 ./mvnw -pl reader -am -P pitest verify -DskipITs # Footer, Trailer, PostscriptParser,
-                                                 # SegmentSpec, Layout, FlatSegmentDecoder
+                                                 # SegmentSpec, Layout, SerializedArrayDecoder
 ./mvnw -pl writer -am -P pitest verify -DskipITs # ChunkImpl, WriteRegistry
 ```
 
diff --git a/reader/pom.xml b/reader/pom.xml
@@ -90,7 +90,7 @@
 								<param>io.github.dfa1.vortex.reader.PostscriptParser</param>
 								<param>io.github.dfa1.vortex.reader.SegmentSpec</param>
 								<param>io.github.dfa1.vortex.reader.layout.Layout</param>
-								<param>io.github.dfa1.vortex.reader.FlatSegmentDecoder</param>
+								<param>io.github.dfa1.vortex.reader.SerializedArrayDecoder</param>
 							</targetClasses>
 						</configuration>
 					</plugin>
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/ScanIterator.java b/reader/src/main/java/io/github/dfa1/vortex/reader/ScanIterator.java
@@ -410,7 +410,7 @@ private List<ArrayStats> decodeZoneTable(String column) {
         long nZones = statsFlat.rowCount();
         SegmentSpec spec = file.footer().segmentSpecs().get(segIdx);
         try (Arena tableArena = Arena.ofConfined()) {
-            Array decoded = file.decodeFlatSegment(spec, statsDtype, nZones, tableArena);
+            Array decoded = file.decodeSegment(spec, statsDtype, nZones, tableArena);
             if (!(decoded instanceof StructArray table)) {
                 return null;
             }
@@ -818,8 +818,8 @@ public Array decodeChild(Layout child, DType dtype) {
         }
 
         @Override
-        public Array decodeFlatSegment(SegmentSpec spec, DType dtype, long rowCount) {
-            return file.decodeFlatSegment(spec, dtype, rowCount, arena);
+        public Array decodeSegment(SegmentSpec spec, DType dtype, long rowCount) {
+            return file.decodeSegment(spec, dtype, rowCount, arena);
         }
 
         @Override
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/SerializedArrayDecoder.java b/reader/src/main/java/io/github/dfa1/vortex/reader/SerializedArrayDecoder.java
@@ -23,7 +23,7 @@
 ///
 /// [ReadRegistry] is pure dispatch; this class owns all file-format knowledge:
 /// FlatBuffer parsing, buffer-offset arithmetic, and encoding-spec lookup.
-final class FlatSegmentDecoder {
+final class SerializedArrayDecoder {
 
     /// Hard cap on array-node recursion depth. The encoded array tree nests through child nodes
     /// (validity, patches, run-ends, dictionary codes/values, …); a crafted or self-referential
@@ -37,7 +37,7 @@ final class FlatSegmentDecoder {
     /// Creates a decoder backed by the given registry.
     ///
     /// @param registry the registry used to dispatch to concrete decoder impls
-    public FlatSegmentDecoder(ReadRegistry registry) {
+    public SerializedArrayDecoder(ReadRegistry registry) {
         this.registry = registry;
     }
 
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/VortexHandle.java b/reader/src/main/java/io/github/dfa1/vortex/reader/VortexHandle.java
@@ -32,7 +32,7 @@ public interface VortexHandle extends Closeable {
     /// @param rowCount number of logical rows in the segment
     /// @param arena    allocator for decode output; lifetime matches the caller's chunk epoch
     /// @return the decoded array
-    Array decodeFlatSegment(SegmentSpec spec, DType dtype, long rowCount, SegmentAllocator arena);
+    Array decodeSegment(SegmentSpec spec, DType dtype, long rowCount, SegmentAllocator arena);
 
     /// Returns a read-only view of the bytes backing the given segment spec.
     /// Writes through the returned segment throw `UnsupportedOperationException`.
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/VortexHttpReader.java b/reader/src/main/java/io/github/dfa1/vortex/reader/VortexHttpReader.java
@@ -288,13 +288,13 @@ public long fileSize() {
     // ── HTTP helpers ──────────────────────────────────────────────────────────
 
     @Override
-    public io.github.dfa1.vortex.reader.array.Array decodeFlatSegment(
+    public io.github.dfa1.vortex.reader.array.Array decodeSegment(
             io.github.dfa1.vortex.reader.SegmentSpec spec,
             DType dtype, long rowCount,
             java.lang.foreign.SegmentAllocator arenaOut
     ) {
         MemorySegment seg = rawSegment(spec);
-        return new FlatSegmentDecoder(registry)
+        return new SerializedArrayDecoder(registry)
                 .decode(seg, footer.arraySpecs(), dtype, rowCount, arenaOut);
     }
 
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/VortexReader.java b/reader/src/main/java/io/github/dfa1/vortex/reader/VortexReader.java
@@ -317,13 +317,13 @@ private ArrayStats readFlatStats(Layout flat) {
     }
 
     @Override
-    public io.github.dfa1.vortex.reader.array.Array decodeFlatSegment(
+    public io.github.dfa1.vortex.reader.array.Array decodeSegment(
             io.github.dfa1.vortex.reader.SegmentSpec spec,
             DType dtype, long rowCount,
             java.lang.foreign.SegmentAllocator arena
     ) {
         MemorySegment seg = IoBounds.slice(fileSegment, spec.offset(), spec.length()).asReadOnly();
-        return new FlatSegmentDecoder(registry)
+        return new SerializedArrayDecoder(registry)
                 .decode(seg, footer.arraySpecs(), dtype, rowCount, arena);
     }
 
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/layout/FlatLayoutDecoder.java b/reader/src/main/java/io/github/dfa1/vortex/reader/layout/FlatLayoutDecoder.java
@@ -22,6 +22,6 @@ public Array decode(LayoutDecodeContext ctx, Layout layout, DType dtype) {
         }
         int segIdx = layout.segments().getFirst();
         SegmentSpec spec = ctx.segmentSpec(segIdx);
-        return ctx.decodeFlatSegment(spec, dtype, layout.rowCount());
+        return ctx.decodeSegment(spec, dtype, layout.rowCount());
     }
 }
diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/layout/LayoutDecodeContext.java b/reader/src/main/java/io/github/dfa1/vortex/reader/layout/LayoutDecodeContext.java
@@ -31,7 +31,7 @@ public interface LayoutDecodeContext {
     /// @param dtype    logical type of the decoded array
     /// @param rowCount number of logical rows in the segment
     /// @return the decoded [Array]
-    Array decodeFlatSegment(SegmentSpec spec, DType dtype, long rowCount);
+    Array decodeSegment(SegmentSpec spec, DType dtype, long rowCount);
 
     /// Resolves a segment index (as stored on a flat [Layout]) to its [SegmentSpec] in the
     /// file's segment table.
diff --git a/reader/src/test/java/io/github/dfa1/vortex/reader/ArrayNodeDepthBombSecurityTest.java b/reader/src/test/java/io/github/dfa1/vortex/reader/ArrayNodeDepthBombSecurityTest.java
@@ -15,10 +15,10 @@
 import static org.assertj.core.api.Assertions.assertThatThrownBy;
 
 /// Adversarial tests for the encoded-array-tree recursion in
-/// [FlatSegmentDecoder]'s `convertArrayNode`.
+/// [SerializedArrayDecoder]'s `convertArrayNode`.
 ///
 /// The decoder walks the array node tree recursively (validity, patches, run-ends, dictionary
-/// codes/values, …). Without the [FlatSegmentDecoder#MAX_ARRAY_TREE_DEPTH] cap a crafted segment
+/// codes/values, …). Without the [SerializedArrayDecoder#MAX_ARRAY_TREE_DEPTH] cap a crafted segment
 /// with thousands of nested children produces a [StackOverflowError] — an `Error` that escapes the
 /// "malformed input must surface as [VortexException]" contract (ADR 0003).
 ///
@@ -29,7 +29,7 @@ class ArrayNodeDepthBombSecurityTest {
 
     private static final DType DTYPE = DType.I32;
 
-    private final FlatSegmentDecoder sut = new FlatSegmentDecoder(ReadRegistry.empty());
+    private final SerializedArrayDecoder sut = new SerializedArrayDecoder(ReadRegistry.empty());
 
     @Test
     void arrayTreeAtDepthLimit_clearsGuard() {
@@ -38,7 +38,7 @@ void arrayTreeAtDepthLimit_clearsGuard() {
             // depth == limit there, and `limit > limit` is false. The walk completes and only then
             // fails because the empty registry has no decoder. Kills `depth >` relaxed to `>=`,
             // which would wrongly reject this legal max-depth tree with the depth message.
-            byte[] fb = deeplyNestedArrayFlatBuffer(FlatSegmentDecoder.MAX_ARRAY_TREE_DEPTH);
+            byte[] fb = deeplyNestedArrayFlatBuffer(SerializedArrayDecoder.MAX_ARRAY_TREE_DEPTH);
             MemorySegment seg = wrapAsSegment(fb, arena);
 
             // When / Then
@@ -53,7 +53,7 @@ void arrayTreeOneOverDepthLimit_throwsVortexException() {
         try (Arena arena = Arena.ofConfined()) {
             // Given — one level deeper: the deepest node reaches limit + 1, tripping the guard
             // before any StackOverflowError can escape.
-            byte[] fb = deeplyNestedArrayFlatBuffer(FlatSegmentDecoder.MAX_ARRAY_TREE_DEPTH + 1);
+            byte[] fb = deeplyNestedArrayFlatBuffer(SerializedArrayDecoder.MAX_ARRAY_TREE_DEPTH + 1);
             MemorySegment seg = wrapAsSegment(fb, arena);
 
             // When / Then — must surface as VortexException, not StackOverflowError
diff --git a/reader/src/test/java/io/github/dfa1/vortex/reader/SerializedArrayBoundsSecurityTest.java b/reader/src/test/java/io/github/dfa1/vortex/reader/SerializedArrayBoundsSecurityTest.java
@@ -21,12 +21,12 @@
 /// Both the trailing length field and each buffer descriptor's offset/length come straight
 /// from untrusted file bytes. After ADR 0003 Phase E, every malformed value must surface as
 /// a [VortexException], never a raw `IndexOutOfBoundsException` from `MemorySegment.asSlice`.
-class FlatSegmentBoundsSecurityTest {
+class SerializedArrayBoundsSecurityTest {
 
 
     private static final DType DTYPE = DType.I32;
 
-    private final FlatSegmentDecoder sut = new FlatSegmentDecoder(ReadRegistry.empty());
+    private final SerializedArrayDecoder sut = new SerializedArrayDecoder(ReadRegistry.empty());
 
     @Test
     void segmentSmallerThanLengthField_throwsVortexException() {
diff --git a/reader/src/test/java/io/github/dfa1/vortex/reader/SerializedArrayDecoderDecodeTest.java b/reader/src/test/java/io/github/dfa1/vortex/reader/SerializedArrayDecoderDecodeTest.java
@@ -17,20 +17,20 @@
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.catchThrowable;
 
-/// Successful flat-segment decode path — complements [FlatSegmentBoundsSecurityTest] (which only
+/// Successful flat-segment decode path — complements [SerializedArrayBoundsSecurityTest] (which only
 /// drives the rejection paths). A buffer descriptor with non-zero padding exercises the offset
 /// walk, and an unknown encoding id exercises the unknown-id passthrough through an
 /// allow-unknown registry. Together these pin two otherwise-untested spots:
 /// - the `dataOffset += padding` accumulation: with padding > 0, flipping `+=` to `-=` slices at a
 ///   negative offset and fails, so a clean decode proves the addition.
 /// - the unknown-id node construction: mishandling an unresolvable id there yields a
 ///   node the decode would not turn into an `UnknownArray`.
-class FlatSegmentDecoderDecodeTest {
+class SerializedArrayDecoderDecodeTest {
 
     @Test
     void decode_unknownEncodingWithBufferPadding_returnsUnknownArray() {
         ReadRegistry registry = ReadRegistry.builder().allowUnknown().build();
-        FlatSegmentDecoder sut = new FlatSegmentDecoder(registry);
+        SerializedArrayDecoder sut = new SerializedArrayDecoder(registry);
 
         try (Arena arena = Arena.ofConfined()) {
             // Given — a flat segment whose single buffer carries 8 bytes of leading padding and
@@ -56,7 +56,7 @@ void decode_unknownEncodingWithBufferPadding_returnsUnknownArray() {
     @Test
     void decode_blankEncodingId_throwsVortexException() {
         ReadRegistry registry = ReadRegistry.builder().allowUnknown().build();
-        FlatSegmentDecoder sut = new FlatSegmentDecoder(registry);
+        SerializedArrayDecoder sut = new SerializedArrayDecoder(registry);
 
         try (Arena arena = Arena.ofConfined()) {
             // Given — a zero-length FlatBuffer string in the spec table decodes to "", which
diff --git a/writer/src/test/java/io/github/dfa1/vortex/writer/WriterZoneMapTest.java b/writer/src/test/java/io/github/dfa1/vortex/writer/WriterZoneMapTest.java
@@ -108,7 +108,7 @@ void zoneMaps_statsPayloadDecodesPerZoneMinMax(@TempDir Path tmp) throws IOExcep
             Layout zonesFlat = reader.layout().children().get(0).children().get(1);
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
-                StructArray stats = (StructArray) reader.decodeFlatSegment(spec, numericStatsTableDtype(), 3, arena);
+                StructArray stats = (StructArray) reader.decodeSegment(spec, numericStatsTableDtype(), 3, arena);
                 LongArray max = (LongArray) ((MaskedArray) stats.field("max")).inner();
                 LongArray min = (LongArray) ((MaskedArray) stats.field("min")).inner();
                 LongArray sum = (LongArray) ((MaskedArray) stats.field("sum")).inner();
@@ -153,7 +153,7 @@ void zoneMaps_nullableColumn_recordsPerZoneNullCount(@TempDir Path tmp) throws I
             Layout zonesFlat = reader.layout().children().get(0).children().get(1);
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
-                StructArray stats = (StructArray) reader.decodeFlatSegment(spec, numericStatsTableDtype(), 2, arena);
+                StructArray stats = (StructArray) reader.decodeSegment(spec, numericStatsTableDtype(), 2, arena);
                 LongArray nullCount = (LongArray) ((MaskedArray) stats.field("null_count")).inner();
                 LongArray sum = (LongArray) ((MaskedArray) stats.field("sum")).inner();
 
@@ -259,7 +259,7 @@ void utf8Column_emitsStringMinMaxZoneMap(@TempDir Path tmp) throws IOException {
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
                 StructArray stats =
-                        (StructArray) reader.decodeFlatSegment(spec, utf8StatsTableDtype(), 2, arena);
+                        (StructArray) reader.decodeSegment(spec, utf8StatsTableDtype(), 2, arena);
                 VarBinArray max = (VarBinArray) ((MaskedArray) stats.field("max")).inner();
                 VarBinArray min = (VarBinArray) ((MaskedArray) stats.field("min")).inner();
                 assertThat(min.getString(0)).isEqualTo("apple");
@@ -296,7 +296,7 @@ void extensionColumn_emitsStorageMinMaxZoneMap(@TempDir Path tmp) throws IOExcep
             Layout zonesFlat = column.children().get(1);
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
-                StructArray stats = (StructArray) reader.decodeFlatSegment(spec, statsTableDtype(), 2, arena);
+                StructArray stats = (StructArray) reader.decodeSegment(spec, statsTableDtype(), 2, arena);
                 LongArray max = (LongArray) ((MaskedArray) stats.field("max")).inner();
                 LongArray min = (LongArray) ((MaskedArray) stats.field("min")).inner();
                 assertThat(min.getLong(0)).isEqualTo(10);
@@ -335,7 +335,7 @@ void dictColumn_emitsStringMinMaxZoneMapWrappingDict(@TempDir Path tmp) throws I
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
                 StructArray stats =
-                        (StructArray) reader.decodeFlatSegment(spec, utf8StatsTableDtype(), 2, arena);
+                        (StructArray) reader.decodeSegment(spec, utf8StatsTableDtype(), 2, arena);
                 VarBinArray max = (VarBinArray) ((MaskedArray) stats.field("max")).inner();
                 VarBinArray min = (VarBinArray) ((MaskedArray) stats.field("min")).inner();
                 assertThat(min.getString(0)).isEqualTo("a");
@@ -371,7 +371,7 @@ void primitiveDictColumn_emitsNumericMinMaxZoneMapWrappingDict(@TempDir Path tmp
             Layout zonesFlat = column.children().get(1);
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
-                StructArray stats = (StructArray) reader.decodeFlatSegment(spec, numericStatsTableDtype(), 2, arena);
+                StructArray stats = (StructArray) reader.decodeSegment(spec, numericStatsTableDtype(), 2, arena);
                 LongArray max = (LongArray) ((MaskedArray) stats.field("max")).inner();
                 LongArray min = (LongArray) ((MaskedArray) stats.field("min")).inner();
                 LongArray sum = (LongArray) ((MaskedArray) stats.field("sum")).inner();
@@ -410,7 +410,7 @@ void zoneMaps_f64StatsPayloadDecodesPerZoneMinMaxSum(@TempDir Path tmp) throws I
             Layout zonesFlat = reader.layout().children().get(0).children().get(1);
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
-                StructArray stats = (StructArray) reader.decodeFlatSegment(spec, f64StatsTableDtype(), 3, arena);
+                StructArray stats = (StructArray) reader.decodeSegment(spec, f64StatsTableDtype(), 3, arena);
                 DoubleArray max = (DoubleArray) ((MaskedArray) stats.field("max")).inner();
                 DoubleArray min = (DoubleArray) ((MaskedArray) stats.field("min")).inner();
                 DoubleArray sum = (DoubleArray) ((MaskedArray) stats.field("sum")).inner();
@@ -446,7 +446,7 @@ void zoneMaps_perTypeStatsDecodePerZoneMinMax(PType ptype, @TempDir Path tmp) th
             Layout zonesFlat = reader.layout().children().get(0).children().get(1);
             SegmentSpec spec = reader.footer().segmentSpecs().get(zonesFlat.segments().getFirst());
             try (Arena arena = Arena.ofConfined()) {
-                StructArray stats = (StructArray) reader.decodeFlatSegment(spec, perTypeStatsTableDtype(ptype), 2, arena);
+                StructArray stats = (StructArray) reader.decodeSegment(spec, perTypeStatsTableDtype(ptype), 2, arena);
                 MemorySegment min = ((MaskedArray) stats.field("min")).inner().materialize(arena);
                 MemorySegment max = ((MaskedArray) stats.field("max")).inner().materialize(arena);
                 assertThat(readStat(min, ptype, 0)).as("min zone 0").isEqualTo(0.0);

Original file line number	Diff line number	Diff line change
`@@ -410,7 +410,7 @@ private List<ArrayStats> decodeZoneTable(String column) {`
`410`	`410`	`long nZones = statsFlat.rowCount();`
`411`	`411`	`SegmentSpec spec = file.footer().segmentSpecs().get(segIdx);`
`412`	`412`	`try (Arena tableArena = Arena.ofConfined()) {`
`413`		`- Array decoded = file.decodeFlatSegment(spec, statsDtype, nZones, tableArena);`
	`413`	`+ Array decoded = file.decodeSegment(spec, statsDtype, nZones, tableArena);`
`414`	`414`	`if (!(decoded instanceof StructArray table)) {`
`415`	`415`	`return null;`
`416`	`416`	`}`
`@@ -818,8 +818,8 @@ public Array decodeChild(Layout child, DType dtype) {`
`818`	`818`	`}`
`819`	`819`
`820`	`820`	`@Override`
`821`		`- public Array decodeFlatSegment(SegmentSpec spec, DType dtype, long rowCount) {`
`822`		`- return file.decodeFlatSegment(spec, dtype, rowCount, arena);`
	`821`	`+ public Array decodeSegment(SegmentSpec spec, DType dtype, long rowCount) {`
	`822`	`+ return file.decodeSegment(spec, dtype, rowCount, arena);`
`823`	`823`	`}`
`824`	`824`
`825`	`825`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -317,13 +317,13 @@ private ArrayStats readFlatStats(Layout flat) {`
`317`	`317`	`}`
`318`	`318`
`319`	`319`	`@Override`
`320`		`- public io.github.dfa1.vortex.reader.array.Array decodeFlatSegment(`
	`320`	`+ public io.github.dfa1.vortex.reader.array.Array decodeSegment(`
`321`	`321`	`io.github.dfa1.vortex.reader.SegmentSpec spec,`
`322`	`322`	`DType dtype, long rowCount,`
`323`	`323`	`java.lang.foreign.SegmentAllocator arena`
`324`	`324`	`) {`
`325`	`325`	`MemorySegment seg = IoBounds.slice(fileSegment, spec.offset(), spec.length()).asReadOnly();`
`326`		`- return new FlatSegmentDecoder(registry)`
	`326`	`+ return new SerializedArrayDecoder(registry)`
`327`	`327`	`.decode(seg, footer.arraySpecs(), dtype, rowCount, arena);`
`328`	`328`	`}`
`329`	`329`
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,6 @@ public Array decode(LayoutDecodeContext ctx, Layout layout, DType dtype) {`
`22`	`22`	`}`
`23`	`23`	`int segIdx = layout.segments().getFirst();`
`24`	`24`	`SegmentSpec spec = ctx.segmentSpec(segIdx);`
`25`		`- return ctx.decodeFlatSegment(spec, dtype, layout.rowCount());`
	`25`	`+ return ctx.decodeSegment(spec, dtype, layout.rowCount());`
`26`	`26`	`}`
`27`	`27`	`}`