Skip to content

Conversation

@xxubai
Copy link
Contributor

@xxubai xxubai commented Sep 11, 2025

Why are the changes needed?

Close #3749 .

Brief change log

  • Reduce I/O frequency by tuning manifest operations
  • Limit the maximum number of cached data/delete files to 10k to avoid OOM
  • Expire files partition by partition (starting with older partitions first). This makes physical cleanup during snapshot expiration more readable and effective.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@github-actions github-actions bot added the module:ams-server Ams server module label Sep 11, 2025
@xxubai xxubai force-pushed the manifest-driven-data-expiration branch from b0fc030 to b65ec60 Compare September 11, 2025 15:44
@xxubai xxubai requested a review from zhoujinsong September 12, 2025 12:06
@github-actions
Copy link

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Oct 13, 2025
@xxubai xxubai removed the stale label Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module:ams-server Ams server module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Improvement]: Manifest-driven and partition-aware data expiration

1 participant