[
https://issues.apache.org/jira/browse/CASSANDRA-21199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18086412#comment-18086412
]
Matt Byrd commented on CASSANDRA-21199:
---------------------------------------
[^2026-06-05_125315-mbyrd_CASSANDRA-21199_trunk_ci_summary.html] new CI now we
have
tests failing in both the latest two runs:
{code:java}
┌──────────────────────┬──────────────────────────────────────────────────────┬──────────────────────────────────────────────────────────────┐
│ Suite │ Class
│ Method │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm-dtest-simulator │ simulator.test.AccordHarrySimulationTest
│ test-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm-dtest-simulator │ simulator.test.EpochStressTest
│ manyEpochsAndAccordConverges-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm-dtest-simulator │ simulator.test.ShortAccordSimulationTest
│ simulationTest-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm-dtest-simulator │ simulator.test.SingleNodeSingleTableASTTest
│ accordFull-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm-dtest-simulator │ simulator.test.SingleNodeSingleTableASTTest
│ accordMixedReads-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm-upgrade-dtests │ distributed.upgrade.ClusterMetadataUpgradeHarryTest
│ mappertimedout │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm11-dtests │ distributed.test.accord.AccordMigrationTest
│ testPaxosToAccordSerialRead-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm11-dtests │ distributed.test.accord.InteropAccordCQLTest
│ testCASSimulatorLite-_jdk11 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ jvm21-dtests │ distributed.test.accord.InteropAccordCQLTest
│ testCASSimulatorLite-_jdk21 │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ python-dtests-latest │ bootstrap_test.TestBootstrap
│ test_consistent_range_movement_false_with_rf1_should_succeed │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ python-dtests-latest │ bootstrap_test.TestBootstrap
│ test_consistent_range_movement_true_with_rf1_should_fail │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ python-dtests-latest │ client_request_metrics_test.TestClientRequestMetrics
│ test_client_request_metrics │
├──────────────────────┼──────────────────────────────────────────────────────┼──────────────────────────────────────────────────────────────┤
│ python-dtests-latest │ replace_address_test.TestReplaceAddress
│ test_replace_with_insufficient_replicas │
└──────────────────────┴──────────────────────────────────────────────────────┴──────────────────────────────────────────────────────────────┘
{code}
seem mostly unrelated?
CC [~marcuse]
> ProtocolVersion::supportedVersions can be computed once to avoid showing up
> in allocation profiles
> --------------------------------------------------------------------------------------------------
>
> Key: CASSANDRA-21199
> URL: https://issues.apache.org/jira/browse/CASSANDRA-21199
> Project: Apache Cassandra
> Issue Type: Improvement
> Components: Local/Other
> Reporter: Matt Byrd
> Assignee: Matt Byrd
> Priority: Normal
> Fix For: 6.x, 7.x
>
> Attachments:
> 026-06-01_114157-mbyrd_CASSANDRA-21199_trunk_ci_summary.html,
> 2026-06-05_125315-mbyrd_CASSANDRA-21199_trunk_ci_summary.html,
> CASSANDRA-21394_mbyrd_trunk_ci_summary.html, Screenshot 2026-03-04 at 7.01.51
> PM.png
>
>
> It's a relatively small amount in one allocation profile (see attached
> screenshot) ~0.3% of allocations, but still seems un-necessary as we're
> copying elements from a static final EnumSet
> {code}
> /** All supported versions, published as an enumset */
> public final static EnumSet<ProtocolVersion> SUPPORTED =
> EnumSet.copyOf(Arrays.asList(ArrayUtils.addAll(SUPPORTED_VERSIONS)));
> {code}
> Could instead just compute this in static initialization .
> {code}
> public static List<String> supportedVersions()
> {
> List<String> ret = new ArrayList<>(SUPPORTED.size());
> for (ProtocolVersion version : SUPPORTED)
> ret.add(version.toString());
> return ret;
> }
> {code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
