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/05/02 10:31:32 UTC

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

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

   Thanks for the review @kou !  I have added a new `oracle-java` entry on the `docker-compose.yml` and I have updated the `debian-java` to use the `maven:*` images directly removing the wrapper image `ci/docker/debian-9-java.dockerfile`, as you said I can't find anywhere where the `ARROW_JAVA_SHADE_FLATBUFS` were being used. I have also removed the `JDK_NAME` and have added a JDK 8 CI test (as it seems to be the default defined on our .env).
   
   As per the failure on JDK 17 it is expected until we merge https://github.com/apache/arrow/pull/12941, the idea is to rebase once that PR is merged and validate the JDK 17 image is passing at that point!


-- 
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