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

Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2020/12/19 01:40:38 UTC

[GitHub] [incubator-ratis] runzhiwang edited a comment on pull request #364: [WIP]RATIS-1248. Support network topology

runzhiwang edited a comment on pull request #364:
URL: https://github.com/apache/incubator-ratis/pull/364#issuecomment-748396823


   @szetszwo Hi, sorry, now I have one problem, where should I put the route table in proto ? Now I put route table in DataStreamInitProto, and encode DataStreamInitProto as  RaftClientRequestProto#message. But this can not work for filestore, because filestore encode FileStoreRequestProto as RaftClientRequestProto#message, I can only put route table in StreamWriteRequestProto. But this seems a little complicated, maybe we should put route table in a same proto, no matter whether it's filestore, to achieve this, maybe we should put Raft.proto and Examples.proto together. Do you have any suggestion ?


----------------------------------------------------------------
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:
users@infra.apache.org