Skip to content

Commit

Permalink
fix: JS should modify data in redirected space, not key space
Browse files Browse the repository at this point in the history
  • Loading branch information
niloc132 committed Sep 19, 2024
1 parent 45ebf06 commit ae460a2
Showing 1 changed file with 13 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit ae460a2

Please sign in to comment.