XComp commented on pull request #19047: URL: https://github.com/apache/flink/pull/19047#issuecomment-1065076645
We agreed on going for the log-warning approach because we don't want to cause a failover of the Flink cluster in such a case where the job is already marked as clean. Flink behaves as expected. The user should only be informed about it through logs. Additionally, we can improve the change by moving the check into `Dispatcher.submitJob` because that's called by all the different execution modes. I reverted the initial commits and pushed the changes to the remote branch. PTAL -- 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]
