arvindKandpal-ksolves opened a new pull request, #4598:
URL: https://github.com/apache/cassandra/pull/4598
This PR fixes a test failure in `CompactionHistorySystemTableUpgradeTest`
introduced by recent changes to compaction history.
**The Issue:**
The upgrade test asserts that legacy compaction history records (from nodes
before the upgrade) should contain the string `"UNKNOWN"`. However, for records
missing the new compaction properties, the system currently outputs an empty
map representation `"{}"`.
**CI Failure Example:**
java.lang.AssertionError: Output did not contain expected data. Expected KS:
distributed_test_keyspace Table: tb Type: UNKNOWN Actual Output: ... {} ...
**The Fix:**
Updated the test assertion in `CompactionHistorySystemTableUpgradeTest.java`
to expect `"{}"` instead of `"UNKNOWN"` for legacy records, matching the actual
output.
patch by Arvind Kandpal; reviewed by TBD for CASSANDRA-21151
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
