[
https://issues.apache.org/jira/browse/RATIS-210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16402118#comment-16402118
]
Mukul Kumar Singh commented on RATIS-210:
-----------------------------------------
Thanks for the contribution [~szetszwo]. I have committed this patch to master.
> Refactor client request proto
> -----------------------------
>
> Key: RATIS-210
> URL: https://issues.apache.org/jira/browse/RATIS-210
> Project: Ratis
> Issue Type: Improvement
> Reporter: Tsz Wo Nicholas Sze
> Assignee: Tsz Wo Nicholas Sze
> Priority: Major
> Fix For: 0.2.0-alpha
>
> Attachments: r210_20180315.patch
>
>
> In RATIS-208, we plan to add a parameter for client specifying the number of
> required servers in a write request. In the current code, it is hard (or
> ugly) to add such parameter.
> In this JIRA, we propose to replace enum Type in RaftClientRequestProto by
> {code}
> oneof Type {
> WriteRequestProto write = 3;
> ReadRequestProto read = 4;
> StaleReadRequestProto staleRead = 5;
> }
> {code}
> so that each proto can carry the request specific parameters.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
