Skip to content
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.

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

Upgrade latest cassandra version to 4.1 #14564

Closed

Conversation

krvikash
Copy link
Contributor

@krvikash krvikash commented Oct 11, 2022

Description

Upgrade latest cassandra version to 4.1

Non-technical explanation

NA

Release notes

( ) This is not user-visible or docs only and no release notes are required.
( ) Release notes are required, please propose a release note for me.
(X) Release notes are required, with the following suggested text:

# Section
* Upgrade latest cassandra version to 4.1. ({issue}`14564`)

@krvikash
Copy link
Contributor Author

Cassandra 4.1 container is not coming up and thus tests are failing.

Looks like Thrift and some properties have been removed from 4.0+. We need to modify the cassandra.yaml

https://github.com/apache/cassandra/blob/cassandra-4.1/test/unit/org/apache/cassandra/config/ConfigCompatabilityTest.java#L65-L90

@krvikash krvikash force-pushed the upgrade-cassandra-version-to-4.1 branch from 07c49e7 to 1e86bea Compare October 12, 2022 12:23
@krvikash
Copy link
Contributor Author

I have modified the existing plugin/trino-cassandra/src/test/resources/cu-cassandra.yaml file with the contents of https://github.com/apache/cassandra/blob/cassandra-4.1/conf/cassandra.yaml.

I have kept most of the property's values according to the previous revision. commitlog_total_space and commitlog_segment_size value is the latest from the 4.1 version.

@krvikash
Copy link
Contributor Author

Due to common changes for upgrading version 3.0 and 4.1. Moving this change to #14562. Closing this PR

@krvikash krvikash closed this Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

5 participants