Skip to content

Commit 71817cb

Browse files
committed
Revert one of the checks added in BooleanSparseArraySource
1 parent e569c5d commit 71817cb

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

engine/table/src/main/java/io/deephaven/engine/table/impl/sources/BooleanSparseArraySource.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ public void setNull(long key) {
106106
return;
107107
}
108108
final int indexWithinBlock = (int) (key & INDEX_MASK);
109-
if (BooleanUtils.isNull(blocks2[indexWithinBlock])) {
109+
if (blocks2[indexWithinBlock] == NULL_BOOLEAN_AS_BYTE) {
110110
return;
111111
}
112112

replication/static/src/main/java/io/deephaven/replicators/ReplicateSourcesAndChunks.java

+1-3
Original file line numberDiff line numberDiff line change
@@ -1163,9 +1163,7 @@ private static void replicateBooleanSparseArraySource() throws IOException {
11631163
lines = simpleFixup(lines, "nullByRanges",
11641164
"block\\[indexWithinBlock\\] != NULL_BOOLEAN", "!BooleanUtils.isNull(block[indexWithinBlock])",
11651165
"NULL_BOOLEAN", "NULL_BOOLEAN_AS_BYTE");
1166-
lines = simpleFixup(lines, "setNull",
1167-
"blocks2\\[indexWithinBlock\\] == NULL_BOOLEAN", "BooleanUtils.isNull(blocks2[indexWithinBlock])",
1168-
"NULL_BOOLEAN", "NULL_BOOLEAN_AS_BYTE");
1166+
lines = simpleFixup(lines, "setNull", "NULL_BOOLEAN", "NULL_BOOLEAN_AS_BYTE");
11691167

11701168
lines = replaceRegion(lines, "copyFromTypedArray", Arrays.asList(
11711169
" for (int jj = 0; jj < length; ++jj) {",

0 commit comments

Comments
 (0)