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

[Cassandra 4.x migration Step 2] Use org.apache.cassandra:cassandra-all 4.0.11 #174

Merged
merged 5 commits into from
Jan 9, 2024

Conversation

Jinn-Niam
Copy link
Contributor

@Jinn-Niam Jinn-Niam commented Oct 4, 2023

Need to merge #177 first, then we can merge this PR to use Cassandra 4.x.

  • Use org.apache.cassandra:cassandra-all 4.0.11.
  • Remove invalid configuration from cassandra.yaml (May need to revisit the 'cassandra.yaml' in the future, since in this PR we basically remove deprecated configuration from cassandra.yaml of Cassandra 3.x. We may also need to check the cassandra.yaml of Cassandra 4.x )

@Jinn-Niam Jinn-Niam changed the title Switch to Cassandra 4.0 [WIP] Switch to Cassandra 4.0 Oct 4, 2023
@Jinn-Niam Jinn-Niam marked this pull request as draft October 24, 2023 00:21
@Jinn-Niam Jinn-Niam changed the title [WIP] Switch to Cassandra 4.0 [Block] Switch to Cassandra 4.0 Oct 24, 2023
@Jinn-Niam Jinn-Niam changed the title [Block] Switch to Cassandra 4.0 [Cassandra 4.x migration Step 2] Use org.apache.cassandra:cassandra-all 4.0.11 Nov 5, 2023
@Jinn-Niam Jinn-Niam marked this pull request as ready for review November 5, 2023 02:17
@slachiewicz
Copy link
Member

#177 has been merged with 452a932

before I merge this PR - one question, why do we need still cassandra-thrift dependency?

@Jinn-Niam
Copy link
Contributor Author

#177 has been merged with 452a932

before I merge this PR - one question, why do we need still cassandra-thrift dependency?

Thanks for reminding me. As you said, the cassandra-thrift dependency is no longer needed since we are using cassandra 4.x.

@slachiewicz slachiewicz merged commit dec3c7d into mojohaus:master Jan 9, 2024
3 checks passed
@slachiewicz
Copy link
Member

Thank You

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants