Skip to content

Commit 1998cea

Browse files
dfa1claude
andcommitted
fix(performance): use Array.materialize() in benchmark after ArraySegments.of() removal
The vortex-performance module (Linux/macOS-only CI step, skipped on Windows) still referenced the removed ArraySegments.of(). Swap to arr.materialize(arena). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 60fce2e commit 1998cea

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

performance/src/main/java/io/github/dfa1/vortex/performance/RustWritesJavaReadsBigFileBenchmark.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
import dev.vortex.arrow.ArrowAllocation;
1010
import dev.vortex.jni.NativeLoader;
1111
import io.github.dfa1.vortex.reader.array.Array;
12-
import io.github.dfa1.vortex.reader.array.ArraySegments;
1312
import io.github.dfa1.vortex.reader.ReadRegistry;
1413
import io.github.dfa1.vortex.reader.VortexReader;
1514
import io.github.dfa1.vortex.reader.Chunk;
@@ -181,7 +180,7 @@ private long scanJava() throws IOException {
181180
while (iter.hasNext()) {
182181
try (Chunk c = iter.next()) {
183182
Array arr = c.columns().get("c0");
184-
MemorySegment buf = ArraySegments.of(arr, Arena.ofAuto());
183+
MemorySegment buf = arr.materialize(Arena.ofAuto());
185184
long count = buf.byteSize() / Long.BYTES;
186185
for (long i = 0; i < count; i++) {
187186
sum += buf.getAtIndex(LE_LONG, i);

0 commit comments

Comments
 (0)