Skip to content

Conversation

adelapena
Copy link

Only count live data size when we are using bytes-sized limits. This prevents issues with paging when row purging or other unexpected transformation changes the size of the read rows.

@adelapena adelapena self-assigned this Sep 22, 2025
Copy link

github-actions bot commented Sep 22, 2025

Checklist before you submit for review

  • This PR adheres to the Definition of Done
  • Make sure there is a PR in the CNDB project updating the Converged Cassandra version
  • Use NoSpamLogger for log lines that may appear frequently in the logs
  • Verify test results on Butler
  • Test coverage for new/modified code is > 80%
  • Proper code formatting
  • Proper title for each commit staring with the project-issue number, like CNDB-1234
  • Each commit has a meaningful description
  • Each commit is not very long and contains related changes
  • Renames, moves and reformatting are in distinct commits
  • All new files should contain the DataStax copyright header instead of the Apache License one

@adelapena adelapena force-pushed the CNDB-15435-main-dont-count branch from 6cacbf6 to 7fbd8e5 Compare September 23, 2025 10:25
Only count live data size when we are using bytes-sized limits.
This prevents issues with paging when row purging or other unexpected
transformation changes the size of the read rows.

See DBPE-16935 and DBPE-17751.
@adelapena adelapena force-pushed the CNDB-15435-main-dont-count branch from 7fbd8e5 to 0cd4aa4 Compare September 24, 2025 10:52
Copy link

@cassci-bot
Copy link

❌ Build ds-cassandra-pr-gate/PR-2010 rejected by Butler


2 regressions found
See build details here


Found 2 new test failures

Test Explanation Runs Upstream
o.a.c.distributed.test.UnableToParseClientMessageTest.badHeader[version=4/v4] REGRESSION 🔴🔵 0 / 5
o.a.c.net.ProxyHandlerConnectionsTest.suddenDisconnect (compression) REGRESSION 🔴🔵 0 / 5

Found 2 known test failures

@adelapena adelapena merged commit d8005b7 into main Sep 24, 2025
482 of 492 checks passed
@adelapena adelapena deleted the CNDB-15435-main-dont-count branch September 24, 2025 11:45
michaelsembwever pushed a commit that referenced this pull request Sep 25, 2025
…ng (#2010)

Only count live data size when we are using bytes-sized limits. This
prevents issues with paging when row purging or other unexpected
transformation changes the size of the read rows.

This fixes  DBPE-16935 and DBPE-17751.

Rebase notes:
 * nowInSec and minDeletionTime are not longs instead of ints (ref CASSANDRA-14227)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants