Gabriel39 commented on PR #68162: URL: https://github.com/apache/doris/pull/68162#issuecomment-5727616659
Fixed both JDBC URL concatenations in the Lance prepared-search regression. A leading `+` on a new line was parsed by Groovy as unary plus on a String, causing the runtime `String.positive()` error even though script compilation succeeded. Verified locally before committing: - Built the current PR FE with `build.sh --fe`. - Ran the original complete suite and reproduced the same `String.positive()` failure. - Ran the fixed complete suite against a master FE, an observer FE, one BE, and MinIO/Lance fixtures: **1 suite passed, 0 failures, 0 skips**. - All **12 actual ServerPreparedStatement executions** passed, covering direct and forwarded execution, indexed and flat search, and repeated A/B/A parameter binding. Both connections retained `emulateUnsupportedPstmts=false`. The fix does not weaken any assertions or skip forwarding coverage. -- 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]
