failure

Author: wuchong

.github/workflows/ci.yaml

@@ -37,7 +37,7 @@ jobs:
       - name: Test
         timeout-minutes: 60
         run: |
-          mvn -B verify -Ptest-coverage -Dlog.dir=${{ runner.temp }}/fluss-logs -Dlog4j.configurationFile=${{ github.workspace }}/tools/ci/log4j.properties || true
+          mvn -B verify -Ptest-coverage -Dlog.dir=${{ runner.temp }}/fluss-logs -Dlog4j.configurationFile=${{ github.workspace }}/tools/ci/log4j.properties
         env:
           MAVEN_OPTS: -Xmx4096m
           ARTIFACTS_OSS_ENDPOINT: ${{ secrets.ARTIFACTS_OSS_ENDPOINT }}
@@ -49,6 +49,7 @@ jobs:
           ARTIFACTS_OSS_ROLE_ARN: ${{ secrets.ARTIFACTS_OSS_ROLE_ARN }}
       - name: Upload build logs
         uses: actions/upload-artifact@v4
+        if: ${{ failure() }}
         with:
           name: maven-build-logs
           path: ${{ runner.temp }}/fluss-logs/*

fluss-test-utils/src/main/java/com/alibaba/fluss/testutils/common/TestLoggerExtension.java

@@ -49,7 +49,7 @@ public void testSuccessful(ExtensionContext context) {
         LOG.info(
                 "\n--------------------------------------------------------------------------------"
                         + "\nTest {}.{}[{}] successfully run."
-                        + "\n================================================================================",
+                        + "\n================================================================================\n",
                 context.getRequiredTestClass().getCanonicalName(),
                 context.getRequiredTestMethod().getName(),
                 context.getDisplayName());
@@ -60,7 +60,7 @@ public void testFailed(ExtensionContext context, Throwable cause) {
         LOG.error(
                 "\n--------------------------------------------------------------------------------"
                         + "\nTest {}.{}[{}] failed with:\n{}"
-                        + "\n================================================================================",
+                        + "\n================================================================================\n",
                 context.getRequiredTestClass().getCanonicalName(),
                 context.getRequiredTestMethod().getName(),
                 context.getDisplayName(),