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

Reset repairedAt when receiving sstables #27

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

dmitry-potepalov
Copy link

We're not using incremental repairs, therefore we want Repaired At to be set to 0 for all sstables. Some older services still have this value set to non-zero values from the times when we were using incremental repairs. This patch should make sure that any new node we bootstrap has Repaired At set to 0 for all sstables.

This patch should be removed when we release the next minor version after 4.1.7.

We're not using incremental repairs, therefore we want Repaired At to
be set to 0 for all sstables. Some older services still have this value
set to non-zero values from the times when we were using incremental
repairs. This patch should make sure that any new node we bootstrap
has Repaired At set to 0 for all sstables.

This patch should be removed when we release the next minor version
after 4.1.7.
Copy link

github-actions bot commented Nov 12, 2024

Unit Test Results

7 145 tests  ±0   7 091 ✔️ ±0   1h 28m 16s ⏱️ +33s
   863 suites ±0        54 💤 ±0 
   863 files   ±0          0 ±0 

Results for commit 925fe27. ± Comparison against base commit 59987dc.

♻️ This comment has been updated with latest results.

@dmitry-potepalov dmitry-potepalov requested a review from a team November 12, 2024 11:49
@aris-aiven aris-aiven merged commit 87e11bf into aiven-4.1.7 Nov 14, 2024
3 checks passed
@aris-aiven aris-aiven deleted the dmitry-potepalov-rewrite-repaired-at branch November 14, 2024 11:55
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