Oxidaner opened a new pull request, #1096: URL: https://github.com/apache/incubator-seata-go/pull/1096
## What this PR does This PR completes the concrete code-level bug fixes identified in #879. The follow-up investigation items mentioned later in #879 are broader in scope and too large to reasonably combine into the same change, so this PR only focuses on the directly actionable bug fixes. ## What is included - return `beforeCommit()` errors in XA/AT commit paths - extend AT SQL AST traversal coverage - harden `util/log` error logging - clarify `SQLHook` error-handling behavior - make AT executors consistently return `beforeHooks()` errors - add targeted regression tests ## What is still left The broader follow-up items from #879 still remain, mainly: - Getty communication follow-up - full future lifecycle validation - reconnect behavior under network instability - monitoring / alerting for abnormal session close paths - broader stress and failure-path test coverage These should be handled separately in follow-up work rather than bundled into this PR. ## Verification Ran: - `go test ./pkg/datasource/sql/exec/at ./pkg/datasource/sql/exec` -- 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]
