[
https://issues.apache.org/jira/browse/RATIS-2601?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18095679#comment-18095679
]
ryu1.sakai commented on RATIS-2601:
-----------------------------------
I will open a GitHub pull request soon. I have already made a fix with a unit
test.
> GroupInfoReply conf is lost when converting to GroupInfoReplyProto
> ------------------------------------------------------------------
>
> Key: RATIS-2601
> URL: https://issues.apache.org/jira/browse/RATIS-2601
> Project: Ratis
> Issue Type: Bug
> Components: client
> Reporter: ryu1.sakai
> Priority: Major
>
> ClientProtoUtils#toGroupInfoReplyProto() doesn't set the field 'conf'
> (RaftConfigurationProto) when it converts GroupInfoReply into
> GroupInfoReplyProto.
> Consequently, the converted proto lacks conf and GroupInfoReply.getConf()
> always returns an empty Optional on the client side.
> Note: Although this code is located in the client module (ratis-client), it
> is used on the server side (GrpcAdminProtocolService and NettyRpcService), so
> the bug causes incorrect behavior on the service side.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
