yujun777 commented on PR #63884: URL: https://github.com/apache/doris/pull/63884#issuecomment-4590547972
Updated the PR to align `COM_RESET_CONNECTION` behavior with MySQL more completely. Changes added in the latest commit: - Preserve the current catalog/database across reset connection. - Reset user variables, prepared statements, running query state, insert result, command state, temporary tables, and transaction state. - Return reset cleanup errors instead of silently returning OK. - Return autocommit server status on reset OK. - Use the MySQL-compatible unknown prepared statement error after reset clears prepared statements. - Fixed the FE test import ordering issue from review. Validation: - `tools/fast-compile-fe.sh` - `tools/fast-compile-fe.sh --test ConnectContextTest.java` - `run-fe-ut.sh --run ConnectContextTest` - `mvn -f fe/pom.xml -pl fe-core -am validate -DskipTests -Dmaven.build.cache.enabled=false` - `run-regression-test.sh --run -d query_p0/session_variable -s test_reset_connection_session_variable` -- 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]
