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

Rebase our patches on top of cassandra-4.1.7 #26

Merged
merged 4 commits into from
Nov 12, 2024

Conversation

dmitry-potepalov
Copy link

Before we can deploy 4.1.7, we need to rebase existing patches on top of cassandra-4.1.7. I've created the aiven-4.1.7 branch pointing at the upstream 4.1.7, this PR just rebases the patches we applied on top of 4.1.5.

The data will be provided by Astacus, streaming from other nodes is not
necessary.
It's more convenient to set it there in Astacus. Option semantics is
preserved, the value specified in Java properties has priority.
We decorate standard PasswordAuthenticator with a few platform specific
features, that don't change the externally observable behavior - so it's
absolutely compatible with PasswordAuthenticator.

A few tools and client libraries do not support custom authenticators.
For example: ZDM Proxy. They fail whenever they see anything but
PasswordAuthenticator nevertheless they are in fact completely
compatible.

So we want to stop advertising our custom authenticator and pretend that
it's a standard PasswordAuthenticator.
@dmitry-potepalov dmitry-potepalov requested a review from a team November 11, 2024 13:37
Copy link

github-actions bot commented Nov 11, 2024

Unit Test Results

7 145 tests   7 091 ✔️  1h 27m 43s ⏱️
   863 suites       54 💤
   863 files           0

Results for commit 59987dc.

♻️ This comment has been updated with latest results.

Copy link

@aris-aiven aris-aiven left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@aris-aiven aris-aiven merged commit 516b245 into aiven-4.1.7 Nov 12, 2024
3 checks passed
@aris-aiven aris-aiven deleted the dmitry-potepalov-cassandra-deploy-417 branch November 12, 2024 09:10
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.

3 participants