Generate docs with JNI

Author: kou

.github/workflows/rc.yml

@@ -76,49 +76,6 @@ jobs:
           name: release-source
           path: |
             apache-arrow-java-*
-  docs:
-    name: Docs
-    runs-on: ubuntu-latest
-    timeout-minutes: 30
-    needs:
-      - source
-    env:
-      DOCKER_VOLUME_PREFIX: .docker/
-    permissions:
-      contents: read
-      packages: write
-    steps:
-      - name: Download source archive
-        uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
-        with:
-          name: release-source
-      - name: Extract source archive
-        run: |
-          tar -xf apache-arrow-java-*.tar.gz --strip-components=1
-      - uses: docker/login-action@9780b0c442fbb1117ed29e0efdff1e18412f7567 # v3.3.0
-        with:
-          registry: ghcr.io
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
-      - name: Cache
-        uses: actions/cache@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0
-        with:
-          path: .docker
-          key: docs-${{ hashFiles('compose.yaml', '**/pom.xml', '**/*.java') }}
-          restore-keys: docs-
-      - name: Build
-        run: |
-          docker compose run \
-            --volume "${PWD}/build/:/build/" \
-            docs
-      - name: Compress into single artifact to keep directory structure
-        run: |
-          tar -cvzf docs.tar.gz -C build/java docs
-      - name: Upload artifacts
-        uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0
-        with:
-          name: release-docs-${{ matrix.platform.arch }}
-          path: docs.tar.gz
   jni-linux:
     name: JNI ${{ matrix.platform.runs_on }} ${{ matrix.platform.arch }}
     runs-on: ${{ matrix.platform.runs_on }}
@@ -143,6 +100,14 @@ jobs:
       contents: read
       packages: write
     steps:
+      - name: Prepare
+        run: |
+          if [ ${{ matrix.platform.arch }} = "x86_64" ]; then
+            ARROW_JAVA_BUILD_DOCS=ON
+          else
+            ARROW_JAVA_BUILD_DOCS=ON
+          fi
+          echo "ARROW_JAVA_BUILD_DOCS=${ARROW_JAVA_BUILD_DOCS}" >> ${GIHUTB_ENV}
       - name: Download source archive
         uses: actions/download-artifact@fa0a91b85d4f404e444e00e005971372dc801d16 # v4.1.8
         with:
@@ -176,7 +141,10 @@ jobs:
           restore-keys: jni-linux-${{ matrix.platform.arch }}-
       - name: Build
         run: |
-          docker compose run vcpkg-jni
+          docker compose run \
+            --env ARROW_JAVA_BUILD_DOCS=${ARROW_JAVA_BUILD_DOCS} \
+            --volume "${PWD}/build/:/build/" \
+            vcpkg-jni
       - name: Push Docker image
         if: success() && github.event_name == 'push' && github.repository == 'apache/arrow-java' && github.ref_name == 'main'
         run: |
@@ -188,6 +156,15 @@ jobs:
         with:
           name: jni-linux-${{ matrix.platform.arch }}
           path: jni-linux-${{ matrix.platform.arch }}.tar.gz
+      - name: Compress docs
+        if: env.ARROW_JAVA_BUILD_DOCS == 'docs'
+        run: tar -cvzf docs.tar.gz -C build/java docs
+      - name: Upload artifacts
+        if: env.ARROW_JAVA_BUILD_DOCS == 'docs'
+        uses: actions/upload-artifact@65c4c4a1ddee5b72f698fdd19549f0f0fb45cf08 # v4.6.0
+        with:
+          name: release-docs-${{ matrix.platform.arch }}
+          path: docs.tar.gz
   jni-macos:
     name: JNI ${{ matrix.platform.runs_on }} ${{ matrix.platform.arch }}
     runs-on: ${{ matrix.platform.runs_on }}

ci/scripts/test.sh

@@ -23,10 +23,14 @@ if [[ "${ARROW_JAVA_TEST:-ON}" != "ON" ]]; then
   exit
 fi
 
-source_dir=${1}
+source_dir="$(cd "${1}" && pwd)"
 build_dir=${2}/java
 java_jni_dist_dir=${3}
 
+if [ -d "${java_jni_dist_dir}" ]; then
+  java_jni_dist_dir="$(cd "${java_jni_dist_dir}" && pwd)"
+fi
+
 mvn="mvn -B -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn"
 # Use `2 * ncores` threads
 mvn="${mvn} -T 2C"

compose.yaml

@@ -50,28 +50,6 @@ services:
         /arrow-java/ci/scripts/build.sh /arrow-java /build /jni &&
         /arrow-java/ci/scripts/test.sh /arrow-java /build /jni"
 
-  docs:
-    # Build docs.
-    #
-    # Usage:
-    #   docker compose build docs
-    #   docker compose run docs
-    # Parameters:
-    #   MAVEN: 3.9.9
-    #   JDK: 11, 17, 21
-    image: ${ARCH}/maven:${MAVEN}-eclipse-temurin-${JDK}
-    volumes:
-      - .:/arrow-java:delegated
-      - ${DOCKER_VOLUME_PREFIX}maven-cache:/root/.m2:delegated
-    environment:
-      ARROW_JAVA_BUILD_DOCS: "ON"
-      ARROW_JAVA_SKIP_GIT_PLUGIN: "ON"
-    command:
-      - /arrow-java/ci/scripts/build.sh
-      - /arrow-java
-      - /build
-      - /jni
-
   conda-jni-cdata:
     # Builds and tests just the C Data Interface JNI library and JARs.
     # (No dependencies on arrow-cpp.)