arvindKandpal-ksolves opened a new pull request, #4594: URL: https://github.com/apache/cassandra/pull/4594
This PR fixes a CI regression introduced by CASSANDRA-20081. **Context:** The `CompactionHistoryTest#testCompactionProperties` test asserts that the compaction strategy in the `nodetool compactionhistory` output matches `SizeTieredCompactionStrategy` (STCS). However, in the CI environment (trunk), the default compaction strategy is often randomized or defaults to `UnifiedCompactionStrategy` (UCS). This causes an assertion failure because the test creates the table without an explicit strategy, leading to a mismatch between expected (STCS) and actual (UCS) output. **CI Failure Example:** https://ci-cassandra.apache.org/job/Cassandra-trunk/2387/#showFailuresLink Expected Props: {strategy=SizeTieredCompactionStrategy, ...} Actual Output: {strategy=UnifiedCompactionStrategy, ...} **The Fix:** Explicitly enforce `SizeTieredCompactionStrategy` in the test's `CREATE TABLE` statement. This ensures deterministic output regardless of the environment's default configuration. patch by Arvind Kandpal; reviewed by TBD for CASSANDRA-21145 -- 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]
