StephanEwen commented on pull request #14890: URL: https://github.com/apache/flink/pull/14890#issuecomment-778856699
I think this PR is fine. It means that the `reset()` method of the Master Hooks is always invoked, also on partial failover and on starting a new job. Given that this is an internal API with Pravega as the only user I am aware of, I think this change is no problem. So +1 to merge this. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
