You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by "XComp (via GitHub)" <gi...@apache.org> on 2023/04/18 11:26:33 UTC

[GitHub] [flink] XComp commented on pull request #22380: [FLINK-31773][runtime] Separate DefaultLeaderElectionService.start(LeaderContender) into two separate methods for starting the driver and registering a contender

XComp commented on PR #22380:
URL: https://github.com/apache/flink/pull/22380#issuecomment-1512910996

   Thanks for the comments, @zentol and @reswqa . I addressed them. @zentol raised some valid concerns about the thread-safety which I missed in this PR. I decided to create FLINK-31878 and cover the thread-safety issue separately before going ahead with this (and the subsequent) PR(s).


-- 
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: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org