From dcf7324968ba7c2ccb4660debb650afcc035145b Mon Sep 17 00:00:00 2001 From: Davide Angelocola Date: Sat, 24 Oct 2026 22:08:12 +0200 Subject: [PATCH] chore(reader): suppress java:S6218 on SelectedArrays views Ten array-carrying records in SelectedArrays fail the new-code reliability condition of the quality gate. Same standard suppression as the other records; #602 removes these views altogether. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01SwhK4W5cgj7y7s4ZPCif5x --- .../dfa1/vortex/reader/array/SelectedArrays.java | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/reader/src/main/java/io/github/dfa1/vortex/reader/array/SelectedArrays.java b/reader/src/main/java/io/github/dfa1/vortex/reader/array/SelectedArrays.java index c8e7e936..2cd54995 100644 --- a/reader/src/main/java/io/github/dfa1/vortex/reader/array/SelectedArrays.java +++ b/reader/src/main/java/io/github/dfa1/vortex/reader/array/SelectedArrays.java @@ -188,6 +188,7 @@ private static long offsetAt(Array offsets, long i) { }; } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Longs(LongArray inner, long[] rows) implements LongArray { @Override public long length() { @@ -205,6 +206,7 @@ public long getLong(long i) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Ints(IntArray inner, long[] rows) implements IntArray { @Override public long length() { @@ -222,6 +224,7 @@ public int getInt(long i) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Shorts(ShortArray inner, long[] rows) implements ShortArray { @Override public long length() { @@ -248,6 +251,7 @@ public long fold(long identity, LongBinaryOperator op) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Bytes(ByteArray inner, long[] rows) implements ByteArray { @Override public long length() { @@ -274,6 +278,7 @@ public long fold(long identity, LongBinaryOperator op) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Doubles(DoubleArray inner, long[] rows) implements DoubleArray { @Override public long length() { @@ -291,6 +296,7 @@ public double getDouble(long i) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Floats(FloatArray inner, long[] rows) implements FloatArray { @Override public long length() { @@ -308,6 +314,7 @@ public float getFloat(long i) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Float16s(Float16Array inner, long[] rows) implements Float16Array { @Override public long length() { @@ -341,6 +348,7 @@ public MemorySegment materialize(SegmentAllocator arena) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Bools(BoolArray inner, long[] rows) implements BoolArray { @Override public long length() { @@ -358,6 +366,7 @@ public boolean getBoolean(long i) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record VarBins(VarBinArray inner, long[] rows) implements VarBinArray { @Override public long length() { @@ -407,6 +416,7 @@ public VarBinArray limited(long n) { } } + @SuppressWarnings("java:S6218") // internal data carrier; record components are arrays of immutable primitives or refs that flow through pipelines without ever being compared. private record Decimals(DecimalArray inner, long[] rows) implements DecimalArray { @Override public long length() {