You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ratis.apache.org by "Tsz-wo Sze (Jira)" <ji...@apache.org> on 2020/02/04 23:49:00 UTC

[jira] [Commented] (RATIS-759) Support stream APIs to send large messages

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

Tsz-wo Sze commented on RATIS-759:
----------------------------------

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

Yes, it is a good idea, although the original code does not have it.

> 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)