Skip to content

Commit

Permalink
[KYUUBI #1520] [BUILD] Switch to setup-java@v2
Browse files Browse the repository at this point in the history
### _Why are the changes needed?_

Migrate to GA setup-java from v1 to v2

https://github.com/actions/setup-java

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.readthedocs.io/en/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #1520 from pan3793/ga.

Closes #1520

7aad9d2 [Cheng Pan] nit
5b31242 [Cheng Pan] Use setup-java@v2

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Cheng Pan <chengpan@apache.org>
  • Loading branch information
pan3793 committed Dec 7, 2021
1 parent 986d983 commit 627d66b
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 11 deletions.
16 changes: 10 additions & 6 deletions .github/workflows/master.yml
Expand Up @@ -16,9 +16,11 @@ jobs:
runs-on: ubuntu-20.04
steps:
- uses: actions/checkout@v2
- uses: actions/setup-java@v1
- name: Setup JDK 8
uses: actions/setup-java@v2
with:
java-version: '1.8'
distribution: zulu
java-version: 8
- run: build/mvn org.apache.rat:apache-rat-plugin:check
- name: Upload rat report
if: failure()
Expand Down Expand Up @@ -61,8 +63,9 @@ jobs:
steps:
- uses: actions/checkout@v2
- name: Setup JDK ${{ matrix.java }}
uses: actions/setup-java@v1
uses: actions/setup-java@v2
with:
distribution: zulu
java-version: ${{ matrix.java }}
- uses: actions/cache@v1
with:
Expand Down Expand Up @@ -117,10 +120,11 @@ jobs:
SPARK_LOCAL_IP: localhost
steps:
- uses: actions/checkout@v2
- name: Setup JDK 1.8
uses: actions/setup-java@v1
- name: Setup JDK 8
uses: actions/setup-java@v2
with:
java-version: '1.8'
distribution: zulu
java-version: 8
- uses: actions/cache@v1
with:
path: ~/.m2/repository/com
Expand Down
7 changes: 4 additions & 3 deletions .github/workflows/nightly.yml
Expand Up @@ -16,10 +16,11 @@ jobs:
SPARK_LOCAL_IP: localhost
steps:
- uses: actions/checkout@v2
- name: Setup JDK 1.8
uses: actions/setup-java@v1
- name: Setup JDK 8
uses: actions/setup-java@v2
with:
java-version: '1.8'
distribution: zulu
java-version: 8
- uses: actions/cache@v1
with:
path: ~/.m2/repository/com
Expand Down
5 changes: 3 additions & 2 deletions .github/workflows/publish-snapshot.yml
Expand Up @@ -26,9 +26,10 @@ jobs:
key: snapshot-maven-${{ hashFiles('**/pom.xml') }}
restore-keys: |
snapshot-maven-
- name: Install Java 8
uses: actions/setup-java@v1
- name: Setup JDK 8
uses: actions/setup-java@v2
with:
distribution: zulu
java-version: 8
- name: Publish snapshot - master
if: ${{ matrix.branch == 'master' }}
Expand Down

0 comments on commit 627d66b

Please sign in to comment.