Skip to content

fix(storage): bound coverage cleanup batch - #459

Merged
fewensa merged 1 commit into
mainfrom
agt/datalens-cleanup-deadline
Aug 16, 2026
Merged

fix(storage): bound coverage cleanup batch#459
fewensa merged 1 commit into
mainfrom
agt/datalens-cleanup-deadline

Conversation

@fewensa

@fewensa fewensa commented Aug 16, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • reduce coverage-index-v2 cleanup records per compaction tick from 32 to 4
  • keep cleanup safety validation and read-side delta budget unchanged
  • update maintenance tests for the smaller cleanup batch

Production evidence

  • Arbitrum compaction tick started, then did not emit a completed/failed log after cleanup scan
  • the last observed scan had eligible_records=32 max_records=32, so cleanup work could monopolize the tick before hot bucket compaction progressed

Correctness

  • does not raise or disable the read-side 128 delta-object guard
  • does not skip snapshot/head safety checks before deleting old deltas
  • does not delete S3/RustFS objects manually

Tests

  • cargo fmt --check
  • cargo test -p datalens-storage --test maintenance test_compaction_coverage_index_v2 -- --nocapture
  • cargo clippy -p datalens-storage --all-targets -- -D warnings

@fewensa
fewensa merged commit 1ec9687 into main Aug 16, 2026
1 check passed
@fewensa
fewensa deleted the agt/datalens-cleanup-deadline branch August 16, 2026 09:29
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.

1 participant