swuferhong commented on PR #1401: URL: https://github.com/apache/fluss/pull/1401#issuecomment-3431248561
> > > I have a question here. Do we need to properly handle `coordinatorEpoch` and `coordinatorEpochZkVersion` in this PR, just like Kafka does? In my opinion: > > > > > > * `coordinatorEpoch` prevents TabletServers from processing requests from zombie CoordinatorServers in the event of a split-brain scenario among CoordinatorServers. > > > * `coordinatorEpochZkVersion`([KAFKA-6082](https://issues.apache.org/jira/browse/KAFKA-6082)) prevents zombie CoordinatorServers from modifying the ZK state during a split-brain scenario. > > > > > > Should we take these two concerns into consideration in the same way as Kafka? > > > > > > Thanks for reminder. I considered `coordinatorEpoch` and `coordinatorEpochZkVersion `, but ultimately did not implement them in this PR to simplify its complexity. I think we can handle it later. > > Do you have any suggestions? @LB-Yu @wuchong @swuferhong > > IMO, we can split them into different PRs, but they should be merged together? Otherwise, we might introduce other metadata inconsistency issues when introducing HA. What's your opinion? +1 @zcoo Could you pls create these two issues? I think we can split the work between us. -- 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]
