reswqa commented on PR #21137: URL: https://github.com/apache/flink/pull/21137#issuecomment-1296507911
> We might also want to check the other `LeaderContender` implementations (as you already mentioned, `ResourceManagerImpl` does not suffer from this bug but the others might because they don't use this `handleLeaderEventHandler` pattern. @XComp I totally agree that we should also check the other `LeaderContender` to ensure they are not suffer from this. However, from my point of view, in consideration of limiting the scope of this pull request, I suggest that only the `JobMasterServiceLeadershipRunner` be tested in this PR. For other `LeaderContender`, I will open a separate ticket for tracking. WDYT? -- 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]
