HyukjinKwon commented on code in PR #45172:
URL: https://github.com/apache/spark/pull/45172#discussion_r1495104491
##########
.github/workflows/maven_test.yml:
##########
@@ -159,20 +159,22 @@ jobs:
with:
distribution: zulu
java-version: ${{ matrix.java }}
- - name: Install Python 3.9
+ - name: Install Python 3.11
uses: actions/setup-python@v5
# We should install one Python that is higher than 3+ for SQL and Yarn
because:
# - SQL component also has Python related tests, for example,
IntegratedUDFTestUtils.
# - Yarn has a Python specific test too, for example, YarnClusterSuite.
- if: contains(matrix.modules, 'resource-managers#yarn') ||
(contains(matrix.modules, 'sql#core')) || contains(matrix.modules, 'connect')
+ # macos (14) already has its Python installed, see also SPARK-47096 and
+ #
https://github.com/actions/runner-images/blob/main/images/macos/macos-14-Readme.md
+ if: contains(inputs.os, 'ubuntu') && contains(matrix.modules,
'resource-managers#yarn') || (contains(matrix.modules, 'sql#core')) ||
contains(matrix.modules, 'connect')
Review Comment:
```suggestion
if: contains(inputs.os, 'ubuntu') && (contains(matrix.modules,
'resource-managers#yarn') || (contains(matrix.modules, 'sql#core')) ||
contains(matrix.modules, 'connect'))
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
