Skip to content

[GLUTEN-11103][VL] Optimize Celeborn tests#11105

Merged
kerwin-zk merged 1 commit intoapache:mainfrom
kerwin-zk:celeborn-tests
Nov 18, 2025
Merged

[GLUTEN-11103][VL] Optimize Celeborn tests#11105
kerwin-zk merged 1 commit intoapache:mainfrom
kerwin-zk:celeborn-tests

Conversation

@kerwin-zk
Copy link
Copy Markdown
Contributor

What changes are proposed in this pull request?

#11103

How was this patch tested?

CI

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions github-actions bot removed CORE works for Gluten Core BUILD VELOX labels Nov 18, 2025
matrix:
spark: [ "spark-3.2" ]
celeborn: [ "celeborn-0.6.1", "celeborn-0.5.4", "celeborn-0.4.3"]
celeborn: [ "celeborn-0.6.1", "celeborn-0.5.4"]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kerwin-zk, will 0.4.3 be dropped permanently? If so, could you clean up some related code in a separate PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@philo-he The current main version of Celeborn in use is 0.5, so version 0.4 no longer needs to be tested. I'll submit a new PR to clean this up.

Copy link
Copy Markdown
Member

@philo-he philo-he left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix.

@kerwin-zk kerwin-zk merged commit 12c940a into apache:main Nov 18, 2025
50 checks passed
@zhouyuan
Copy link
Copy Markdown
Member

@kerwin-zk thanks for the fix. just want to confirm, is this due to mixing the sort/hash tests in one instance?

@kerwin-zk
Copy link
Copy Markdown
Contributor Author

@kerwin-zk thanks for the fix. just want to confirm, is this due to mixing the sort/hash tests in one instance?

@zhouyuan Currently, log analysis indicates that the primary cause is insufficient disk space, which led to Celeborn worker failures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants