[jira] [Resolved] (KUDU-2110) RPC footer may be appended more than once

["Michael Ho (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/KUDU-2110?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Michael Ho resolved KUDU-2110.
------------------------------
       Resolution: Fixed
    Fix Version/s: 1.5.0

The offending commit was backed out (https://github.com/apache/kudu/commit/55c40fc72241a0569d311bdb2f0268389512a807).

> RPC footer may be appended more than once
> -----------------------------------------
>
>                 Key: KUDU-2110
>                 URL: https://issues.apache.org/jira/browse/KUDU-2110
>             Project: Kudu
>          Issue Type: Bug
>          Components: rpc
>    Affects Versions: 1.5.0
>            Reporter: Michael Ho
>            Assignee: Michael Ho
>            Priority: Blocker
>              Labels: 1.6.0
>             Fix For: 1.5.0
>
>
> The fix for KUDU-2065 included a footer to RPC messages. The footer is appended during the beginning of the transmission of the outbound transfer for an outbound call. However, the check for the beginning of transmission for an outbound call isn't quite correct as it's possible for an outbound transfer to not send anything in Transfer::SendBuffer().
> {noformat}
>     // Transfer for outbound call must call StartCallTransfer() before transmission can
>     // begin to append footer to the payload if the remote supports it.
>     if (!transfer->TransferStarted() &&
>         transfer->is_for_outbound_call() &&
>         !StartCallTransfer(transfer)) {
>       OutboundQueuePopFront();
>       continue;
>     }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)