Repository navigation
chore(deps): update github workflows - #8036
renovate[bot] wants to merge 1 commit into
Conversation
| with: | ||
| distribution: 'temurin' | ||
| java-version: '21' | ||
| java-version: '25' |
There was a problem hiding this comment.
Spark 3.5.9 integration tests inherit Java 25
The Tasks workflow installs Temurin 25, but the tox JAVA_HOME override only matches spark-v3.0.3 and spark-v3.5.6, not the configured spark-v3.5.9 environments. Those Spark 3.5.9 tests therefore run with Java 25, which is outside that Spark line’s supported Java versions and can prevent the Spark context from starting. Keep the job on a supported JDK or add a matching override.
Evidence
test-integrations-tasks.ymlinstalls Temurin 25 before running thepy${{ matrix.python-version }}-sparktox environments.tox.iniconfigures Spark 3.5.9 environments for Python 3.8, 3.10, and 3.11, but its Java override matchesspark-v{3.0.3,3.5.6}and does not apply tospark-v3.5.9.tests/integrations/spark/test_spark.py::create_spark_contextstarts a realSparkContext, so the inherited Java runtime is used by the integration tests.
Identified by Warden · code-review · 4CC-FHB
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 0a04245. Configure here.
| with: | ||
| distribution: 'temurin' | ||
| java-version: '21' | ||
| java-version: '25' |
There was a problem hiding this comment.
Java 25 breaks Spark tests
High Severity
The Tasks workflow now installs Java 25, but Spark tests still start a real SparkContext against pyspark 3.0.3 and 3.5.9. Those releases do not support Java 25 and fail during context startup because Subject.getSubject is unsupported and SecurityManager is disabled. The Tasks job will fail and block the required check.
Reviewed by Cursor Bugbot for commit 0a04245. Configure here.
Coverage Results 📊✅ 133657 passed | ❌ 9 failed | ⏭️ 7276 skipped | Total: 140942 | Pass Rate: 94.83% | Execution Time: 467m 18s 📊 Comparison with Base Branch
✨ Test counts unchanged from base. ✅ Patch coverage is 100.00% (no changed executable lines found; target 80%). Coverage diff@@ Coverage Diff @@
## master #8036 +/-##
==========================================
- Coverage 90.26% 90.23% -0.03%
==========================================
Files 202 202 —
Tracked lines 26660 26660 —
Branches 9926 9926 —
==========================================
- Hits 24063 24055 -8
- Misses 2597 2605 +8
Partials 1510 1510 —Generated by Coverage Action |


This PR contains the following updates:
0.4.3→0.4.40.4.3→0.4.421→25Release Notes
getsentry/coverage-action
v0.4.4Compare Source
New Features ✨
Internal Changes 🔧
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR was generated by Mend Renovate. View the repository job log.