joeyutong commented on PR #1030: URL: https://github.com/apache/flink-agents/pull/1030#issuecomment-5393235140
Thanks for addressing all three comments. I re-reviewed 4edd5f6 and don’t see any further issues with the change itself. The only remaining item is to rebase and resolve the conflicts with current main. When resolving ActionExecutionOperator, please keep notifyActionReused(actionTask) after both memory replays. Once the conflicts are resolved, this looks good to me. -- 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]
