fix: Fix chunk filling loop bug that broke rollup snapshots with view…

…ports bigger than 512 rows (#6205)
deephaven · Oct 16, 2024 · b4ebd67 · b4ebd67
1 parent b4fb03b
commit b4ebd67
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/...able/src/main/java/io/deephaven/engine/table/impl/hierarchical/HierarchicalTableImpl.java b/...able/src/main/java/io/deephaven/engine/table/impl/hierarchical/HierarchicalTableImpl.java
@@ -1473,10 +1473,12 @@ private void fillNormalDestinations(@NotNull final RowSequence rows) {
 
             try (final RowSequence.Iterator rowsIter =
                     rows.size() >= chunkSize ? rows.getRowSequenceIterator() : null) {
-                final RowSequence chunkRows =
-                        rowsIter == null ? rows : rowsIter.getNextRowSequenceWithLength(chunkSize);
-                final int chunkRowsSize = chunkRows.intSize();
                 do {
+                    final RowSequence chunkRows = rowsIter == null
+                            ? rows
+                            : rowsIter.getNextRowSequenceWithLength(chunkSize);
+                    final int chunkRowsSize = chunkRows.intSize();
+                    Assert.leq(chunkRowsSize, "chunkRowsSize", capacity, "capacity");
                     for (int di = 0, ci = columns.nextSetBit(0); ci >= 0; ++di, ci = columns.nextSetBit(ci + 1)) {
                         if (ci == ROW_EXPANDED_COLUMN_INDEX) {
                             continue;