You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/04/28 20:27:08 UTC

[GitHub] [arrow] kou commented on pull request #13021: ARROW-16327: [Java][CI] Add Java 17 to CI matrix for java workflows

kou commented on PR #13021:
URL: https://github.com/apache/arrow/pull/13021#issuecomment-1112624192

   It seems that `maven:3.8.5-openjdk-17` is based on Oracle Linux 8 not Debian:
   
   ```console
   $ docker run --rm maven:3.8.5-openjdk-17 cat /etc/system-release
   Oracle Linux Server release 8.5
   ```
   
   So existing `ci/docker/debian-9-java.dockerfile` filename isn't suitable for OpenJDK 17.
   
   How about adding `ci/docker/oracle-8-java.dockerfile`, adding `oracle-java` to `docker-compose.yml` and removing `jdk_name` from `.env`?
   
   BTW, it seems that nobody uses `ARROW_JAVA_SHADE_FLATBUFS` defined in `ci/docker/debian-9-java.dockerfile`. See also: 7bc2b0f3579b6fc2d1d6d9797f3a1e5e3726131f
   
   We may be able to use `maven:*` images directly instead of defining our (thin) wrapper image. For example, we use existing images directly for `python-wheel-manylinux-test-imports`.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org