[ 
https://issues.apache.org/jira/browse/RATIS-759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17026585#comment-17026585
 ] 

Shashikant Banerjee edited comment on RATIS-759 at 1/31/20 12:27 PM:
---------------------------------------------------------------------

Thanks [~szetszwo] for working on this. The patch looks good and it would be a 
great feature addition to ozone in particular.

I have just one comment :

Do we also need to add handling for LeaderNotReadyException in function 
"exception2RaftClientReply"? 


was (Author: shashikant):
Thanks [~szetszwo] for working on this. The patch looks good and it would be a 
great feature addition to ozone in particular.

I have just one comment :

Do we also need to add handling for LeaderNotReadyException in function 
"exception2RaftClientReply"? I am +1 on the patch otherwise.

> Support stream APIs to send large messages
> ------------------------------------------
>
>                 Key: RATIS-759
>                 URL: https://issues.apache.org/jira/browse/RATIS-759
>             Project: Ratis
>          Issue Type: New Feature
>          Components: client, server
>            Reporter: Tsz-wo Sze
>            Assignee: Tsz-wo Sze
>            Priority: Major
>         Attachments: r759_20200115.patch, r759_20200123.patch
>
>
> It is inefficient to send a large message using 
> send(Message)/sendAsync(Message) in RaftClient.  We already have 
> RaftOutputStream implemented with sendAsync(..).  We propose adding the 
> following new APIs
> {code}
>   /** Create a stream to send a large message. */
>   MessageOutputStream stream();
>   /** Send the given message using a stream. */
>   CompletableFuture<RaftClientReply> streamAsync(Message message);
> {code} 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to