kevingurney commented on PR #38274: URL: https://github.com/apache/arrow/pull/38274#issuecomment-1768573294
@Divyansh200102 - generally speaking, adding code to the MATLAB interface shouldn't require using `docker-compose`. Is there any specific reason you are trying to use this? --- Looking through the [`eclipse-java` part of the `docker-compose.yml` file](https://github.com/apache/arrow/blame/d68f8e21643e8aec9ed253094edfd15c0a08f1c1/docker-compose.yml#L1696) I see the following: ```shell eclipse-java: # Usage: # docker-compose build eclipse-java # docker-compose run eclipse-java # Parameters: # MAVEN: 3.9.4 # JDK: 17, 21 image: ${ARCH}/maven:${MAVEN}-eclipse-temurin-${JDK} shm_size: *shm-size volumes: *java-volumes command: *java-command ``` It looks like the values of the `MAVEN` and `JDK` environment variables that are used in computing the Docker image name are being pulled from the [default `.env` file](https://github.com/apache/arrow/blob/d68f8e21643e8aec9ed253094edfd15c0a08f1c1/.env#L68C8-L68C8). Perhaps you could try modifying these values to `3.9.4` and `21` based on the comments in the `eclipse-java` code block above? You could also try overriding these variables directly on the command line, like: `MAVEN=3.5.4 JDK=21 docker-compose build eclipse-java` Someone who contributes to the Docker / Java parts of the project like @danepitkin might be able to offer more guidance here. Maybe the default values in the `.env` file should be bumped to `3.9.4` and `21`? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
