-
Notifications
You must be signed in to change notification settings - Fork 28.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WIP] Fix scheduled jobs for branch-3.4 & branch-3.5 #45551
Closed
panbingkun
wants to merge
71
commits into
apache:master
from
panbingkun:fix_scheduled_jobs_branch_3.4_3.5
Closed
Changes from 15 commits
Commits
Show all changes
71 commits
Select commit
Hold shift + click to select a range
5189863
[WIP] Fix scheduled jobs for branch-3.4 & branch-3.5
panbingkun f436ca4
test branch-3.5 scheduled
panbingkun 7d3fe21
Revert "test branch-3.5 scheduled"
panbingkun c5615d6
test
panbingkun 3729bf6
fix
panbingkun 1483ba7
fix
panbingkun ad36f71
fix
panbingkun 22f8c20
fix
panbingkun 157436a
fix
panbingkun 4110769
fix
panbingkun 669db8f
fix it
panbingkun 02efe1d
fix it
panbingkun 323d0b3
fix
panbingkun 156ff37
Trigger build
panbingkun b029989
fix it
panbingkun 9958765
fix it
panbingkun 2bc62ec
fix it
panbingkun f8ac6b5
fix it
panbingkun 7ddaa7b
fix it
panbingkun 8b7b6dc
test
panbingkun c5db8d0
fix SCALA_PROFILE
panbingkun f7a3c76
fix
panbingkun f60cb2e
fix
panbingkun eb11a64
Revert "fix"
panbingkun 0d29f7d
Revert "fix"
panbingkun 57fc86f
Revert "fix SCALA_PROFILE"
panbingkun b11d403
fix try
panbingkun b61fad5
fix
panbingkun 3cfabac
test
panbingkun 133b3fc
fix linter
panbingkun 65757ca
hack dev/run-tests
panbingkun 20c245d
hack build_and_test.yml
panbingkun 38cc017
fix
panbingkun 06c783e
fix
panbingkun 3b0c744
hack run_test
panbingkun 8581c2c
pin pandas 2.0.3
panbingkun b892aab
fix python liner
panbingkun 3a151f3
fix python liner
panbingkun 57701c6
fix
panbingkun 7b293e1
fix
panbingkun 8ef4aa3
fix
panbingkun 0e09b9a
Revert "fix"
panbingkun ffd633a
Revert "fix"
panbingkun f6119b9
fix docs build
panbingkun 0afff67
pin Sphinx==3.0.4
panbingkun c2bb18a
fix scikit-learn
panbingkun 1b5e20e
fix markupsafe
panbingkun d7c3c2b
pin alabaster==0.7.12
panbingkun 3934806
final branch-3.5 test
panbingkun c179934
final branch-3.4 test
panbingkun 6683762
Revert "final branch-3.4 test"
panbingkun 94acde5
Final branch-3.5 Jobs test
panbingkun 11fc324
Final branch-3.4 Jobs test
panbingkun 9b1cea5
Final branch-3.5 Jobs test
panbingkun 6137f9b
Final branch-3.4 Jobs test
panbingkun 826947f
Final branch-3.4 Jobs test
panbingkun 1fcc0b1
Final branch-3.4 Jobs test
panbingkun 99a75c9
Final branch-3.4 Jobs test
panbingkun 6669006
Final branch-3.4 Jobs test
panbingkun 18c211c
Final branch-3.4 Jobs test
panbingkun cc29d4e
set PYTHON_TO_TEST: 'python3.9'
panbingkun 48bbee3
test
panbingkun 8a45b47
test
panbingkun a53f8ee
test
panbingkun f5a20d4
test
panbingkun 20dbf0e
branch-3.4
panbingkun d70b07e
test
panbingkun a8d29d0
test
panbingkun 8d564c6
test
panbingkun 53d5477
test
panbingkun a306f5a
matplotlib==3.7.2
panbingkun File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -365,7 +365,7 @@ jobs: | |
pyspark-pandas-connect-part3 | ||
env: | ||
MODULES_TO_TEST: ${{ matrix.modules }} | ||
PYTHON_TO_TEST: 'python3.9' | ||
PYTHON_TO_TEST: '' | ||
HADOOP_PROFILE: ${{ inputs.hadoop }} | ||
HIVE_PROFILE: hive2.3 | ||
GITHUB_PREV_SHA: ${{ github.event.before }} | ||
|
@@ -374,25 +374,19 @@ jobs: | |
SKIP_MIMA: true | ||
SKIP_PACKAGING: true | ||
METASPACE_SIZE: 1g | ||
BRANCH: ${{ inputs.branch }} | ||
"SCALA_PROFILE": "scala2.13", | ||
"ORACLE_DOCKER_IMAGE_NAME": "gvenzl/oracle-xe:21.3.0" | ||
steps: | ||
- name: Checkout Spark repository | ||
uses: actions/checkout@v4 | ||
# In order to fetch changed files | ||
with: | ||
fetch-depth: 0 | ||
repository: apache/spark | ||
ref: ${{ inputs.branch }} | ||
ref: branch-3.5 | ||
- name: Add GITHUB_WORKSPACE to git trust safe.directory | ||
run: | | ||
git config --global --add safe.directory ${GITHUB_WORKSPACE} | ||
- name: Sync the current branch with the latest in Apache Spark | ||
if: github.repository != 'apache/spark' | ||
run: | | ||
echo "APACHE_SPARK_REF=$(git rev-parse HEAD)" >> $GITHUB_ENV | ||
git fetch https://github.com/$GITHUB_REPOSITORY.git ${GITHUB_REF#refs/heads/} | ||
git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' merge --no-commit --progress --squash FETCH_HEAD | ||
git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' commit -m "Merged commit" --allow-empty | ||
# Cache local repositories. Note that GitHub Actions cache has a 10G limit. | ||
- name: Cache Scala, SBT and Maven | ||
uses: actions/cache@v4 | ||
|
@@ -422,7 +416,7 @@ jobs: | |
uses: actions/setup-java@v4 | ||
with: | ||
distribution: zulu | ||
java-version: ${{ matrix.java }} | ||
java-version: 8 | ||
- name: List Python packages (${{ env.PYTHON_TO_TEST }}) | ||
env: ${{ fromJSON(inputs.envs) }} | ||
shell: 'script -q -e -c "bash {0}"' | ||
|
@@ -438,6 +432,14 @@ jobs: | |
curl -s https://repo.anaconda.com/miniconda/Miniconda3-latest-Linux-x86_64.sh > miniconda.sh | ||
bash miniconda.sh -b -p $HOME/miniconda | ||
rm miniconda.sh | ||
- name: Install Python test dependencies for branch-3.4 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In order to |
||
if: inputs.branch == 'branch-3.4' | ||
run: | | ||
python3.9 -m pip install 'numpy==1.24.4' 'pyarrow==12.0.1' 'matplotlib==3.7.2' 'torch==2.0.1' 'torchvision==0.15.2' | ||
- name: Install Python test dependencies for branch-3.5 | ||
if: inputs.branch == 'branch-3.5' | ||
run: | | ||
python3.9 -m pip install 'numpy==1.25.1' 'pyarrow==12.0.1' 'matplotlib==3.7.2' 'torch==2.0.1' 'torchvision==0.15.2' | ||
# Run the tests. | ||
- name: Run tests | ||
env: ${{ fromJSON(inputs.envs) }} | ||
|
@@ -801,53 +803,53 @@ jobs: | |
- java: 21 | ||
os: ubuntu-latest | ||
- java: 21 | ||
os: macos-14 | ||
os: macos-14 | ||
runs-on: ${{ matrix.os }} | ||
timeout-minutes: 300 | ||
steps: | ||
- name: Checkout Spark repository | ||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
repository: apache/spark | ||
ref: ${{ inputs.branch }} | ||
- name: Sync the current branch with the latest in Apache Spark | ||
if: github.repository != 'apache/spark' | ||
run: | | ||
git fetch https://github.com/$GITHUB_REPOSITORY.git ${GITHUB_REF#refs/heads/} | ||
git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' merge --no-commit --progress --squash FETCH_HEAD | ||
git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' commit -m "Merged commit" --allow-empty | ||
- name: Cache Scala, SBT and Maven | ||
uses: actions/cache@v4 | ||
with: | ||
path: | | ||
build/apache-maven-* | ||
build/scala-* | ||
build/*.jar | ||
~/.sbt | ||
key: build-${{ hashFiles('**/pom.xml', 'project/build.properties', 'build/mvn', 'build/sbt', 'build/sbt-launch-lib.bash', 'build/spark-build-info') }} | ||
restore-keys: | | ||
build- | ||
- name: Cache Maven local repository | ||
uses: actions/cache@v4 | ||
with: | ||
path: ~/.m2/repository | ||
key: java${{ matrix.java }}-maven-${{ hashFiles('**/pom.xml') }} | ||
restore-keys: | | ||
java${{ matrix.java }}-maven- | ||
- name: Install Java ${{ matrix.java }} | ||
uses: actions/setup-java@v4 | ||
with: | ||
distribution: zulu | ||
java-version: ${{ matrix.java }} | ||
- name: Build with Maven | ||
run: | | ||
export MAVEN_OPTS="-Xss64m -Xmx2g -XX:ReservedCodeCacheSize=1g -Dorg.slf4j.simpleLogger.defaultLogLevel=WARN" | ||
export MAVEN_CLI_OPTS="--no-transfer-progress" | ||
export JAVA_VERSION=${{ matrix.java }} | ||
# It uses Maven's 'install' intentionally, see https://github.com/apache/spark/pull/26414. | ||
./build/mvn $MAVEN_CLI_OPTS -DskipTests -Pyarn -Pkubernetes -Pvolcano -Phive -Phive-thriftserver -Phadoop-cloud -Djava.version=${JAVA_VERSION/-ea} install | ||
rm -rf ~/.m2/repository/org/apache/spark | ||
- name: Checkout Spark repository | ||
dongjoon-hyun marked this conversation as resolved.
Show resolved
Hide resolved
|
||
uses: actions/checkout@v4 | ||
with: | ||
fetch-depth: 0 | ||
repository: apache/spark | ||
ref: ${{ inputs.branch }} | ||
- name: Sync the current branch with the latest in Apache Spark | ||
if: github.repository != 'apache/spark' | ||
run: | | ||
git fetch https://github.com/$GITHUB_REPOSITORY.git ${GITHUB_REF#refs/heads/} | ||
git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' merge --no-commit --progress --squash FETCH_HEAD | ||
git -c user.name='Apache Spark Test Account' -c user.email='sparktestacc@gmail.com' commit -m "Merged commit" --allow-empty | ||
- name: Cache Scala, SBT and Maven | ||
uses: actions/cache@v4 | ||
with: | ||
path: | | ||
build/apache-maven-* | ||
build/scala-* | ||
build/*.jar | ||
~/.sbt | ||
key: build-${{ hashFiles('**/pom.xml', 'project/build.properties', 'build/mvn', 'build/sbt', 'build/sbt-launch-lib.bash', 'build/spark-build-info') }} | ||
restore-keys: | | ||
build- | ||
- name: Cache Maven local repository | ||
uses: actions/cache@v4 | ||
with: | ||
path: ~/.m2/repository | ||
key: java${{ matrix.java }}-maven-${{ hashFiles('**/pom.xml') }} | ||
restore-keys: | | ||
java${{ matrix.java }}-maven- | ||
- name: Install Java ${{ matrix.java }} | ||
uses: actions/setup-java@v4 | ||
with: | ||
distribution: zulu | ||
java-version: ${{ matrix.java }} | ||
- name: Build with Maven | ||
run: | | ||
export MAVEN_OPTS="-Xss64m -Xmx2g -XX:ReservedCodeCacheSize=1g -Dorg.slf4j.simpleLogger.defaultLogLevel=WARN" | ||
export MAVEN_CLI_OPTS="--no-transfer-progress" | ||
export JAVA_VERSION=${{ matrix.java }} | ||
# It uses Maven's 'install' intentionally, see https://github.com/apache/spark/pull/26414. | ||
./build/mvn $MAVEN_CLI_OPTS -DskipTests -Pyarn -Pkubernetes -Pvolcano -Phive -Phive-thriftserver -Phadoop-cloud -Djava.version=${JAVA_VERSION/-ea} install | ||
rm -rf ~/.m2/repository/org/apache/spark | ||
|
||
# Any TPC-DS related updates on this job need to be applied to tpcds-1g-gen job of benchmark.yml as well | ||
tpcds-1g: | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as the value specified in
branch-3.5.yml