ThreeFire1 commented on PR #18722: URL: https://github.com/apache/dolphinscheduler/pull/18722#issuecomment-6286029224
Thanks, I've simplified the implementation around the current event-dispatch path as suggested. Shared state, metrics, and finalize publication now follow a successful return from `execute()`, with metric failures handled separately. I removed the custom transaction synchronization and the tests for the additional transaction guarantees. Rollback/retry, completion ordering, and real-event-bus serial-queue recovery tests are retained. All 57 selected tests pass on JDK 8 and 11. -- 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]
