incubator-uniffle icon indicating copy to clipboard operation
incubator-uniffle copied to clipboard

[#1711][FOLLOWUP] feat(coordinator): refactor the reconfigurable conf

Open zuston opened this issue 1 year ago • 1 comments

What changes were proposed in this pull request?

Refactor the reconfigurable conf for coordinator side.

Why are the changes needed?

Follow up: #1711

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unit tests

zuston avatar May 24 '24 02:05 zuston

Test Results

 2 433 files  +14   2 433 suites  +14   4h 59m 42s :stopwatch: + 1m 49s    933 tests ± 0     932 :white_check_mark: ± 0   1 :zzz: ±0  0 :x: ±0  10 819 runs  ± 0  10 805 :white_check_mark: ± 0  14 :zzz: ±0  0 :x: ±0 

Results for commit 59ef0f84. ± Comparison against base commit a0e88da5.

This pull request removes 1 and adds 1 tests. Note that renamed tests count towards both.
org.apache.uniffle.test.CoordinatorAssignmentTest ‑ testReconfigureNodeMax
org.apache.uniffle.test.CoordinatorReconfigureNodeMaxTest ‑ testReconfigureNodeMax

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar May 24 '24 03:05 github-actions[bot]

gentle ping @jerqi

zuston avatar May 29 '24 07:05 zuston