From f20bad97e4636ea9abd575f887c5f3d555b8d6cf Mon Sep 17 00:00:00 2001 From: chrislu Date: Sun, 23 Nov 2025 10:15:29 -0800 Subject: [PATCH] fix: force Maven update and verify JAR contains updated code MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Added -U flag to mvn install to force dependency updates Added verification step using javap to check compiled bytecode This will show if the JAR actually contains the new logging code: - If 'totalPosition' string is found → JAR is updated - If not found → Something is wrong with the build The verification output will help diagnose why INFO logs aren't showing. --- .github/workflows/spark-integration-tests.yml | 20 ++++++++++++++++--- 1 file changed, 17 insertions(+), 3 deletions(-) diff --git a/.github/workflows/spark-integration-tests.yml b/.github/workflows/spark-integration-tests.yml index 5c916b6bb..09e4ba8fd 100644 --- a/.github/workflows/spark-integration-tests.yml +++ b/.github/workflows/spark-integration-tests.yml @@ -61,19 +61,33 @@ jobs: run: | echo "Building Java client (required by HDFS clients)..." cd other/java/client - mvn clean install -DskipTests -Dgpg.skip=true -Dcentral.publishing.skip=true + mvn clean install -U -DskipTests -Dgpg.skip=true -Dcentral.publishing.skip=true echo "✓ Java client built and installed to local Maven repo" + + # Verify the JAR contains updated logging code + echo "Verifying JAR contains updated code..." + JAR_PATH=~/.m2/repository/com/seaweedfs/seaweedfs-client/3.80/seaweedfs-client-3.80.jar + if [ -f "$JAR_PATH" ]; then + echo "Checking for updated logging in SeaweedOutputStream..." + if javap -cp "$JAR_PATH" -c seaweedfs.client.SeaweedOutputStream | grep -q "totalPosition"; then + echo "✓ JAR contains updated logging code" + else + echo "⚠️ Warning: JAR may not contain updated code" + echo "Listing class methods:" + javap -cp "$JAR_PATH" seaweedfs.client.SeaweedOutputStream | head -20 + fi + fi cd ../../.. echo "Building HDFS2 client (depends on Java client)..." cd other/java/hdfs2 - mvn clean install -DskipTests -Dgpg.skip=true -Dcentral.publishing.skip=true + mvn clean install -U -DskipTests -Dgpg.skip=true -Dcentral.publishing.skip=true echo "✓ HDFS2 client built" cd ../../.. echo "Building HDFS3 client (depends on Java client)..." cd other/java/hdfs3 - mvn clean install -DskipTests -Dgpg.skip=true -Dcentral.publishing.skip=true + mvn clean install -U -DskipTests -Dgpg.skip=true -Dcentral.publishing.skip=true echo "✓ HDFS3 client built" - name: Prepare artifacts for upload