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

Only run lightweight sync between replicas during restore [DDB-1415] #262

Merged
merged 2 commits into from
Nov 28, 2024

Conversation

aris-aiven
Copy link
Contributor

Lightweight replica sync skips parts merges, part mutations, metadata alter, part cloning from other shards, and pinned part UUID sync. This should be good enough for the cluster to be in a working state, leaving the remaining work to be done asynchronously.

Lightweight replica sync skips parts merges, part mutations, metadata alter,
part cloning from other shards, and pinned part UUID sync. This should be good
enough for the cluster to be in a working state, leaving the remaining work to
be done asynchronously.
@aris-aiven aris-aiven force-pushed the aris-sync-lightweight branch from 26217ed to 3d68892 Compare November 27, 2024 20:08
@aris-aiven aris-aiven requested a review from a team November 27, 2024 20:11
@aris-aiven aris-aiven marked this pull request as ready for review November 27, 2024 20:11
bogdanp05
bogdanp05 previously approved these changes Nov 27, 2024
@aris-aiven aris-aiven requested a review from bogdanp05 November 27, 2024 21:05
@Khatskevich
Copy link
Contributor

Why was it necessary to revert 8ac07a4 ?

@aris-aiven
Copy link
Contributor Author

@Khatskevich it broke integration tests since the setup is trying to connect to Azure over an empty bucket endpoint and times out after 30s

@joelynch joelynch merged commit 973a590 into main Nov 28, 2024
2 checks passed
@joelynch joelynch deleted the aris-sync-lightweight branch November 28, 2024 09:34
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.

4 participants