Commit a84225f
Skip Dataproc shuffle manager auto-configuration (NVIDIA#15420)
Fixes NVIDIA#15418.
### Description
NVIDIA#15285 changed behavior, now
auto set shuffle manager. It impacts the behavior of Dataporc pipeline.
Dataproc's Spark 4.0.1 runtime uses a shuffle resolver ABI that differs
from the Apache Spark ABI expected by the RAPIDS Shuffle Manager. After
shuffle manager auto-configuration was enabled for Spark 4, workloads
that did not explicitly configure `spark.shuffle.manager` began
selecting the RAPIDS implementation and failed during shuffle output
commit with `NoSuchMethodError`.
Skip RAPIDS Shuffle Manager auto-configuration when
`spark.dataproc.engine` is present. This restores the previous behavior
on Dataproc by leaving `spark.shuffle.manager` unset, while continuing
to preserve any explicitly configured shuffle manager. The configuration
documentation is updated to describe this exception.
Added `RapidsPluginUtilsSuite` coverage for the Dataproc guard.
Validation:
- Spark 4.0.1 / Scala 2.13: `RapidsPluginUtilsSuite` passed (6 tests).
- Scalastyle passed across 1,701 files with no errors or warnings.
### Checklists
Documentation
- [x] Updated for new or modified user-facing features or behaviors
- [ ] No user-facing change
Testing
- [x] Added or modified tests to cover new code paths
- [ ] Covered by existing tests
(Please provide the names of the existing tests in the PR description.)
- [ ] Not required
Performance
- [ ] Tests ran and results are added in the PR description
- [ ] Issue filed with a link in the PR description
- [x] Not required
---------
Signed-off-by: Chong Gao <chongg@nvidia.com>
Co-authored-by: Chong Gao <res_life@163.com>1 parent e5bcfff commit a84225f
4 files changed
Lines changed: 17 additions & 4 deletions
File tree
- docs/additional-functionality
- sql-plugin/src
- main/scala/com/nvidia/spark/rapids
- test/scala/com/nvidia/spark/rapids
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | | - | |
| 59 | + | |
60 | 60 | | |
61 | 61 | | |
62 | 62 | | |
| |||
Lines changed: 2 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| 62 | + | |
62 | 63 | | |
63 | 64 | | |
64 | 65 | | |
| 66 | + | |
65 | 67 | | |
66 | 68 | | |
67 | 69 | | |
| |||
Lines changed: 5 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2200 | 2200 | | |
2201 | 2201 | | |
2202 | 2202 | | |
2203 | | - | |
2204 | | - | |
2205 | | - | |
| 2203 | + | |
| 2204 | + | |
| 2205 | + | |
| 2206 | + | |
| 2207 | + | |
2206 | 2208 | | |
2207 | 2209 | | |
2208 | 2210 | | |
| |||
Lines changed: 9 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
69 | 78 | | |
70 | 79 | | |
71 | 80 | | |
| |||
0 commit comments