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

[#1698] fix(test): Increase tests running memory for better stability

Open rickyma opened this issue 1 year ago • 1 comments

What changes were proposed in this pull request?

Increase tests running memory for better stability.

Why are the changes needed?

Fix: https://github.com/apache/incubator-uniffle/issues/1698.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Unnecessary.

rickyma avatar May 20 '24 15:05 rickyma

Test Results

 2 405 files  ±0   2 405 suites  ±0   4h 58m 0s :stopwatch: -18s    931 tests ±0     930 :white_check_mark: ±0   1 :zzz: ±0  0 :x: ±0  10 791 runs  ±0  10 777 :white_check_mark: ±0  14 :zzz: ±0  0 :x: ±0 

Results for commit 7bd9132c. ± Comparison against base commit c3cbdec4.

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

PTAL @zuston @jerqi

rickyma avatar May 21 '24 13:05 rickyma

What's your idea? Gently ping @zuston @jerqi

rickyma avatar May 22 '24 08:05 rickyma

@zuston Have you committed this to branch 0.9?

jerqi avatar May 22 '24 10:05 jerqi

@zuston When you merge a pull request, you should say which branch is merged. like

merged to master & branch 0.9.

jerqi avatar May 22 '24 10:05 jerqi

@zuston When you merge a pull request, you should say which branch is merged. like

merged to master & branch 0.9.

Needn't. Like this, if this PR is cherry-picked to other branches, github will show this.

image

BTW. Some PRs for me is not necessary to cherry-pick to branch-0.9 . If you want this, feel free to tell me advancely

zuston avatar May 23 '24 03:05 zuston

@zuston When you merge a pull request, you should say which branch is merged. like

merged to master & branch 0.9.

Needn't. Like this, if this PR is cherry-picked to other branches, github will show this.

image

BTW. Some PRs for me is not necessary to cherry-pick to branch-0.9 . If you want this, feel free to tell me advancely

If one pull request is a fix, this must be back port to the branch unless the change has great affect. Improvement is optional. We would better to avoid backing port feature.

jerqi avatar May 23 '24 06:05 jerqi