Skip to content

Conversation

@michaelsembwever
Copy link
Member

https://github.com/riptano/cndb/issues/16025

Port into main-5.0 commit 62a1ee1

CNDB-16025: CNDB-15807: Add hints about the size of redacted query values

Add hints about value size when redacting column values in CQL queries printed to logs. No size hints will be included for values smaller than 100 bytes, or for values of fixed-size data types (e.g., int, UUID, timestamp).

Size hints are provided in logarithmic buckets (e.g., ">100B", ">1KiB", ">10KiB", ">100KiB") to give a rough indication of data size while maintaining privacy.

…lues (#2114)

Add hints about value size when redacting column values in CQL queries printed to logs.
No size hints will be included for values smaller than 100 bytes, or for values of
fixed-size data types (e.g., int, UUID, timestamp).

Size hints are provided in logarithmic buckets (e.g., ">100B", ">1KiB", ">10KiB", ">100KiB")
to give a rough indication of data size while maintaining privacy.
@github-actions
Copy link

github-actions bot commented Dec 9, 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

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 9, 2025

Copy link

@adelapena adelapena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@michaelsembwever michaelsembwever merged commit 4b4fd54 into main-5.0 Dec 9, 2025
581 of 594 checks passed
@michaelsembwever michaelsembwever deleted the mck-cndb-16025-main-5.0 branch December 9, 2025 17:17
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.

4 participants