Skip to content

Conversation

@pcmanus
Copy link

@pcmanus pcmanus commented Oct 20, 2025

Updates the file formatting methods for SAI Version.AA to handle the generation used by immutable components, enabling the option to use immutable components with version AA.

This also add a new cassandra.sai.immutable_components.min_version system property that allows to defines the minimum verison starting from which the existing cassandra.sai.immutable_components option will apply. As of this patch, this defaults to ca, because it is the behavior before this patch, but it can change to aa to allow using immutable components with Version.AA.

Fixes https://github.com/riptano/cndb/issues/15745.

@github-actions
Copy link

github-actions bot commented Oct 20, 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

@eolivelli eolivelli requested a review from pkolaczk October 20, 2025 14:21
Updates the file formatting methods for SAI `Version.AA` to handle the
generation used by immutable components, enabling the option to
use immutable components with version AA.

This also add a new `cassandra.sai.immutable_components.min_version`
system property that allows to defines the minimum verison starting from
which the existing `cassandra.sai.immutable_components` option will
apply. As of this patch, this defaults to `ca`, because it is
the behavior before this patch, but it can change to `aa` to
allow using immutable components with `Version.AA`.

Fixes #15745.
@pcmanus pcmanus force-pushed the CNDB-15745-immutable-components-in-AA branch from 69965fa to 70fb738 Compare October 21, 2025 09:15
@sonarqubecloud
Copy link

@cassci-bot
Copy link

✔️ Build ds-cassandra-pr-gate/PR-2079 approved by Butler


Approved by Butler
See build details here

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.

2 participants