fix: JS should modify data in redirected space, not key space (#6085)

Fixes #6084
deephaven · Sep 25, 2024 · 6982984 · 6982984
1 parent 2075615
commit 6982984
Showing 1 changed file with 13 additions and 1 deletion.
diff --git a/...nt-api/src/main/java/io/deephaven/web/client/api/barrage/data/WebBarrageSubscription.java b/...nt-api/src/main/java/io/deephaven/web/client/api/barrage/data/WebBarrageSubscription.java
@@ -366,7 +366,19 @@ public void applyUpdates(WebBarrageMessage message) {
 
                 modifiedColumnSet.set(ii);
 
-                PrimitiveIterator.OfLong destIterator = column.rowsModified.indexIterator();
+                PrimitiveIterator.OfLong destIterator = new PrimitiveIterator.OfLong() {
+                    private final PrimitiveIterator.OfLong wrapped = column.rowsModified.indexIterator();
+
+                    @Override
+                    public long nextLong() {
+                        return redirectedIndexes.get(wrapped.next());
+                    }
+
+                    @Override
+                    public boolean hasNext() {
+                        return wrapped.hasNext();
+                    }
+                };
                 for (int j = 0; j < column.data.size(); j++) {
                     Chunk<Values> chunk = column.data.get(j);
                     destSources[ii].fillChunk(chunk, destIterator);