Skip to content

Commit

Permalink
[SPARK-48840][INFRA] Remove unnecessary existence check for `./dev/fr…
Browse files Browse the repository at this point in the history
…ee_disk_space_container`

### What changes were proposed in this pull request?
This PR removed the check for the existence of `./dev/free_disk_space_container` before execution,  because `./dev/free_disk_space_container` has already been backported to branch-3.4 and branch-3.5 through apache#45624 and apache#43381,  so there is no need to check its existence before execution.

### Why are the changes needed?
Remove unnecessary existence check for `./dev/free_disk_space_container`.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Pass GitHub Actions

### Was this patch authored or co-authored using generative AI tooling?
No

Closes apache#47263 from LuciferYang/SPARK-48840.

Authored-by: yangjie01 <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
  • Loading branch information
LuciferYang authored and jingz-db committed Jul 22, 2024
1 parent ca24fc0 commit 925673e
Showing 1 changed file with 4 additions and 16 deletions.
20 changes: 4 additions & 16 deletions .github/workflows/build_and_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -449,10 +449,7 @@ jobs:
pyspark-coursier-
- name: Free up disk space
shell: 'script -q -e -c "bash {0}"'
run: |
if [ -f ./dev/free_disk_space_container ]; then
./dev/free_disk_space_container
fi
run: ./dev/free_disk_space_container
- name: Install Java ${{ matrix.java }}
uses: actions/setup-java@v4
with:
Expand Down Expand Up @@ -558,10 +555,7 @@ jobs:
restore-keys: |
sparkr-coursier-
- name: Free up disk space
run: |
if [ -f ./dev/free_disk_space_container ]; then
./dev/free_disk_space_container
fi
run: ./dev/free_disk_space_container
- name: Install Java ${{ inputs.java }}
uses: actions/setup-java@v4
with:
Expand Down Expand Up @@ -685,10 +679,7 @@ jobs:
restore-keys: |
docs-maven-
- name: Free up disk space
run: |
if [ -f ./dev/free_disk_space_container ]; then
./dev/free_disk_space_container
fi
run: ./dev/free_disk_space_container
- name: Install Java ${{ inputs.java }}
uses: actions/setup-java@v4
with:
Expand Down Expand Up @@ -831,10 +822,7 @@ jobs:
restore-keys: |
docs-maven-
- name: Free up disk space
run: |
if [ -f ./dev/free_disk_space_container ]; then
./dev/free_disk_space_container
fi
run: ./dev/free_disk_space_container
- name: Install Java ${{ inputs.java }}
uses: actions/setup-java@v4
with:
Expand Down

0 comments on commit 925673e

Please sign in to comment.