xylaaaaa opened a new pull request, #65272:
URL: https://github.com/apache/doris/pull/65272

   ### What problem does this PR solve?
   
   Issue Number: None
   
   Related PR: None
   
   Problem Summary: test_paimon_jdbc_catalog seeds data through Spark before 
Doris reads the Paimon table. The previous shell helper used waitForOrKill, so 
a 300 second Spark seed timeout could continue the case without failing and the 
following Doris query saw an empty table.
   
   The generic Spark ThriftServer JDBC helper is not suitable for this case 
because Paimon's JDBC catalog needs the PostgreSQL driver on the Spark driver 
classpath, and ADD JAR does not make DriverManager find it. This change keeps 
spark-sql for the seed step, passes arguments through ProcessBuilder without 
/bin/bash -c command concatenation, runs spark-sql under an explicit 
container-side timeout, and also fails the suite if the outer process times out 
or exits non-zero.
   
   Spark now uses the container-network MinIO endpoint http://minio:9000 
instead of the host-mapped MinIO port.
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test: Manual test
       - git diff --check --cached
       - ./run-regression-test.sh --run -d external_table_p0/paimon -s 
test_paimon_jdbc_catalog -dryRun passed in the original workspace before moving 
the change to this worktree
       - Attempted the same dryRun in the new worktree; regression framework 
Maven package succeeded, but the script returned 1 at JAVA_UDF_SRC because this 
local environment has no /usr/lib/jvm entry for the script's JDK probe
       - Manual Spark CLI probe for Paimon JDBC catalog with 
--jars/--driver-class-path returned SHOW DATABASES successfully
       - Manual timeout probe confirmed container timeout exits with code 124
   - Behavior changed: Yes (the regression seed command now fails on timeout or 
non-zero exit instead of continuing; product behavior is unchanged)
   - Does this need documentation: No


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to