Skip to content

[GLUTEN-10236][VL] Support both sort and rss_sort shuffle writer for Celeborn#10244

Merged
kerwin-zk merged 7 commits intoapache:mainfrom
marin-ma:extend-shuffle-writer-type
Jul 25, 2025
Merged

[GLUTEN-10236][VL] Support both sort and rss_sort shuffle writer for Celeborn#10244
kerwin-zk merged 7 commits intoapache:mainfrom
marin-ma:extend-shuffle-writer-type

Conversation

@marin-ma
Copy link
Copy Markdown
Contributor

@marin-ma marin-ma commented Jul 22, 2025

By default, use rss sort shuffle implementation for Celeborn. User can set spark.gluten.sql.columnar.shuffle.celeborn.useRssSort=false to use row-based sort implementation.

@marin-ma marin-ma changed the title [GLUTEN-10236][VL]use ShuffleWriterType to replace isSort [GLUTEN-10236][VL] Support both sort and rss_sort shuffle writer for Celeborn Jul 22, 2025
@github-actions
Copy link
Copy Markdown

#10236

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions github-actions Bot added the DOCS label Jul 22, 2025
@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
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions github-actions Bot added the INFRA label Jul 22, 2025
@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@zhouyuan
Copy link
Copy Markdown
Member

Cc @kerwin-zk

@FelixYBW
Copy link
Copy Markdown
Contributor

Could you mention this in https://github.com/apache/incubator-gluten/blob/main/docs/get-started/Velox.md#celeborn-support? There are two set of sort based shuffle implemented and supported in celeborn. Use the config to switch.

How about Uniffle? Does it only support sort based shuffle, right?

@marin-ma
Copy link
Copy Markdown
Contributor Author

How about Uniffle? Does it only support sort based shuffle, right?

@FelixYBW The supported shuffle writer type for Uniffle is same as the local shuffle. We support both hash and row-based sort. We don't use the rss sort shuffle implementation for Uniffle.

@github-actions
Copy link
Copy Markdown

Run Gluten Clickhouse CI on x86

@kerwin-zk kerwin-zk merged commit 804ab4d into apache:main Jul 25, 2025
94 of 95 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants