HyukjinKwon commented on code in PR #45172:
URL: https://github.com/apache/spark/pull/45172#discussion_r1495119252
##########
.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:
Oops yes
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
