Yukang-Lian opened a new pull request, #67222:
URL: https://github.com/apache/doris/pull/67222

   ### What problem does this PR solve?
   
   Issue Number: N/A
   
   Related PR: apache/doris#66289
   
   Problem Summary: SuiteContext.connect() appended Doris mTLS parameters to 
every caller-provided JDBC URL and returned the rewritten string before opening 
a connection. External JDBC setup closures therefore did not run in TLS 
regression jobs. Keep connect() protocol-agnostic and open the exact URL 
supplied by the caller; the default Doris JDBC URL already receives its TLS 
options in Config.createDefaultDb().
   
   ### Release note
   
   None
   
   ### Check List (For Author)
   
   - Test: Unit Test: mvn -Dtest=SuiteContextConnectTest test (1 passed)
   - Behavior changed: Yes. External JDBC connections now preserve their 
supplied URL and execute their setup closure when regression TLS mode is 
enabled.
   - 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