github-actions[bot] commented on code in PR #62698:
URL: https://github.com/apache/doris/pull/62698#discussion_r3233364241


##########
regression-test/conf/regression-conf.groovy:
##########
@@ -24,27 +24,21 @@ defaultDb = "regression_test"
 // init cmd like: select @@session.tx_read_only
 // at each time we connect.
 // add allowLoadLocalInfile so that the jdbc can execute mysql load data from 
client.
-jdbcUrl = 
"jdbc:mysql://127.0.0.1:9030/?useLocalSessionState=true&allowLoadLocalInfile=true&zeroDateTimeBehavior=round"
-targetJdbcUrl = 
"jdbc:mysql://127.0.0.1:9030/?useLocalSessionState=true&allowLoadLocalInfile=true&zeroDateTimeBehavior=round"
+jdbcUrl = 
"jdbc:mysql://127.0.0.1:9330/?useLocalSessionState=true&allowLoadLocalInfile=true&zeroDateTimeBehavior=round"
+targetJdbcUrl = 
"jdbc:mysql://127.0.0.1:9330/?useLocalSessionState=true&allowLoadLocalInfile=true&zeroDateTimeBehavior=round"
 jdbcUser = "root"
 jdbcPassword = ""

Review Comment:
   This looks like a local runner/worktree configuration change accidentally 
committed into the repository defaults. `run-regression-test.sh` uses this file 
as the default regression config, but the PR changes the standard FE ports from 
`9030/9020/8030` to `9330/9220/8330`, removes the CCR defaults, changes 
`max_failure_num` to stop after one failure, and pins 
`pythonUdfRuntimeVersion`. On a normal Doris cluster using the documented 
default ports, the regression suite will connect to the wrong FE and fail 
before it can validate this PR. Please keep 
`regression-test/conf/regression-conf.groovy` at the repository defaults and 
put local port/runtime overrides in a worktree-local or pipeline-specific 
config instead.



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