apache / amoro

Apache Amoro (incubating) is a Lakehouse management system built on open data lake formats.
https://amoro.apache.org/
Apache License 2.0
875 stars 291 forks source link

[AMORO-3265] Refactor SimpleFuture for process api #3266

Closed majin1102 closed 1 month ago

majin1102 commented 1 month ago

Why are the changes needed?

Close #3265 .

Brief change log

  1. Refactor SimpleFuture for pushing process api
  2. Add unit test for SimpleFuture
  3. Move exception package to common module

How was this patch tested?

Documentation

codecov-commenter commented 1 month ago

Codecov Report

Attention: Patch coverage is 91.66667% with 4 lines in your changes missing coverage. Please review.

Project coverage is 22.49%. Comparing base (39a5aa1) to head (8ce7e8f). Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
...in/java/org/apache/amoro/process/SimpleFuture.java 93.61% 2 Missing and 1 partial :warning:
...in/java/org/apache/amoro/process/TableProcess.java 0.00% 1 Missing :warning:

:exclamation: There is a different number of reports uploaded between BASE (39a5aa1) and HEAD (8ce7e8f). Click for more details.

HEAD has 2 uploads less than BASE | Flag | BASE (39a5aa1) | HEAD (8ce7e8f) | |------|------|------| |core|2|0|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #3266 +/- ## ============================================ - Coverage 29.83% 22.49% -7.34% + Complexity 3785 2308 -1477 ============================================ Files 580 397 -183 Lines 47932 38174 -9758 Branches 6210 5439 -771 ============================================ - Hits 14299 8589 -5710 + Misses 32648 28858 -3790 + Partials 985 727 -258 ``` | [Flag](https://app.codecov.io/gh/apache/amoro/pull/3266/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [core](https://app.codecov.io/gh/apache/amoro/pull/3266/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | | [trino](https://app.codecov.io/gh/apache/amoro/pull/3266/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `22.49% <91.66%> (?)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.