Skip to content
Permalink
Browse files
ARROW-16291: [Java] Support JSE17 for Java Cookbooks (#184)
* Initial commit to test Arrow Java with JSE17/18

* Solving problems with SemVer

* Empty-Commit

* Adding actions/setup-python@v3
  • Loading branch information
davisusanibar committed Apr 25, 2022
1 parent 4396e96 commit 6674254808ca2726c7453b12653bbe106eb471dd
Showing 3 changed files with 22 additions and 10 deletions.
@@ -33,16 +33,23 @@ jobs:
test_java_linux:
name: "Test Java Cookbook On Linux"
runs-on: ubuntu-latest
strategy:
matrix:
python-version: [ '3.9' ]
java-version: [ '11', '17', '18' ]
steps:
- uses: actions/checkout@v1
- uses: actions/setup-java@v2
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
with:
python-version: ${{ matrix.python-version }}
- uses: actions/setup-java@v3
with:
distribution: 'temurin'
java-version: '11'
java-version: ${{ matrix.java-version }}
- name: Install dependencies
run: |
sudo apt update
sudo apt install libcurl4-openssl-dev libssl-dev python3-pip maven
sudo apt install libcurl4-openssl-dev libssl-dev
- name: Run tests
run: make javatest
- name: Build cookbook
@@ -33,14 +33,19 @@ jobs:
test_java_osx:
name: "Test Java Cookbook on MacOS"
runs-on: macos-latest
strategy:
matrix:
python-version: [ '3.9' ]
java-version: [ '11', '17', '18' ]
steps:
- uses: actions/checkout@v1
- uses: actions/setup-java@v2
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
with:
python-version: ${{ matrix.python-version }}
- uses: actions/setup-java@v3
with:
distribution: 'temurin'
java-version: '11'
- name: Upgrade pip
run: python3 -m pip install --upgrade pip
java-version: ${{ matrix.java-version }}
- name: Run tests
run: make javatest
- name: Build cookbook
@@ -59,7 +59,7 @@ def compile(
# JDK11 support '-' This allows the pipe to work as expected without requiring a shell
# Migrating to /dev/stdin to also support JDK9+
proc_jshell_process = subprocess.Popen(
["jshell", "--class-path", stdout_dependency, "-s", "/dev/stdin"],
["jshell", "-R--add-opens=java.base/java.nio=ALL-UNNAMED", "--class-path", stdout_dependency, "-s", "/dev/stdin"],
stdin=subprocess.PIPE,
stdout=subprocess.PIPE,
text=True,

0 comments on commit 6674254

Please sign in to comment.