GlenGeng commented on pull request #312: URL: https://github.com/apache/incubator-ratis/pull/312#issuecomment-737071936
@szetszwo Thanks for looking this patch! That would be great to expose getRoleInfoProto() from the new added DivisionInfo. Current development on HDDS-2823 (SCM HA) relies on the public getRoleInfoProto(), so that we can inject term into SCMCommand, and help DN to distinguish a stale leader SCM. After merging back master, we found that we are not able to fetch term from underlying RaftServer. How about merging this walk around right now, and discard it in your RATIS-1192? @runzhiwang will help us to generate a ratis snapshot so that our development of SCM HA won't be blocked. Thanks! ---------------------------------------------------------------- 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]
