Skip to content

[AMORO-4048] Saving cleanup opeartion process info in table_process#4077

Open
zhangwl9 wants to merge 2 commits intoapache:masterfrom
zhangwl9:AMORO-add-clean-histroy-to-optimizeng-info-dev
Open

[AMORO-4048] Saving cleanup opeartion process info in table_process#4077
zhangwl9 wants to merge 2 commits intoapache:masterfrom
zhangwl9:AMORO-add-clean-histroy-to-optimizeng-info-dev

Conversation

@zhangwl9
Copy link
Contributor

@zhangwl9 zhangwl9 commented Feb 6, 2026

Why are the changes needed?

Close #4048.

Brief change log

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? (yes / 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 Feb 6, 2026
@zhangwl9 zhangwl9 force-pushed the AMORO-add-clean-histroy-to-optimizeng-info-dev branch from 84c1f56 to 0c8f0d7 Compare February 6, 2026 11:29
@zhangwl9 zhangwl9 force-pushed the AMORO-add-clean-histroy-to-optimizeng-info-dev branch from f6788dc to 5779284 Compare February 25, 2026 07:17
@codecov-commenter
Copy link

codecov-commenter commented Feb 25, 2026

Codecov Report

❌ Patch coverage is 94.59459% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 28.84%. Comparing base (fda105e) to head (5779284).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
...amoro/server/scheduler/PeriodicTableScheduler.java 94.59% 3 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4077      +/-   ##
============================================
+ Coverage     22.39%   28.84%   +6.44%     
- Complexity     2552     3958    +1406     
============================================
  Files           458      656     +198     
  Lines         42116    52462   +10346     
  Branches       5917     6644     +727     
============================================
+ Hits           9433    15133    +5700     
- Misses        31871    36229    +4358     
- Partials        812     1100     +288     
Flag Coverage Δ
core 28.84% <94.59%> (?)
trino ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

[Feature]: Save the executor cleanup info in table_process

2 participants