New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Align tests with CASSANDRA-19335 #261

Open

zaaath wants to merge 2 commits into apache:trunk from zaaath:CASSANDRA-19335

zaaath commented May 12, 2024 •

edited

Loading

CASSANDRA-19335, introduced in PR#3069, implements the following changes:

By default, nodetool tablestats now outputs byte sizes with human-readable suffixes (e.g., "bytes", "KiB", "MiB"). This update has caused several dtests to fail, as detailed in this CircleCI report.
A new CLI flag, -r or --no-human-readable, has been added to nodetool tablestats to revert to the non-human-readable byte size output, maintaining compatibility with existing parsing logic.

Backwards Compatibility:

Versions of nodetool prior to PR#3069 may not recognize the -r CLI flag, leading to potential test failures.

Further Work:

Implement JSON / YAML output options for nodetool tablestats and update dtests to parse these formats.

zaaath added 2 commits

May 12, 2024 09:36


          Add -r flag to nodetool-tablestats calls (-r stands for raw, or --no-…

3b98523

…human-readable)


          Refactor a call to nodetool tablestats with -r flag

be9495a

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet