This is an automated email from the ASF dual-hosted git repository.

mblow pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git

commit 8c9939184b41cab4d6d075f61fa967255d869f20
Author: Murtadha Hubail <[email protected]>
AuthorDate: Fri Jul 19 02:09:52 2019 +0300

    [NO ISSUE][NET] Do Not Flush Partial Frames on Network Errors
    
    - user model changes: no
    - storage format changes: no
    - interface changes: no
    
    Details:
    - Do not flush partially read frames when a network error
      is reported to avoid processing invalid partial frames
      before the network failure is detected by the channel
      reader.
    
    Change-Id: Ia35b58412cc293426c18a7cd66d8d0a5962db4e2
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/3503
    Sonar-Qube: Jenkins <[email protected]>
    Tested-by: Jenkins <[email protected]>
    Integration-Tests: Jenkins <[email protected]>
    Reviewed-by: Murtadha Hubail <[email protected]>
    Reviewed-by: Till Westmann <[email protected]>
---
 .../org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java   | 1 -
 1 file changed, 1 deletion(-)

diff --git 
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
 
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
index f7ef2aa..75d442f 100644
--- 
a/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
+++ 
b/hyracks-fullstack/hyracks/hyracks-net/src/main/java/org/apache/hyracks/net/protocols/muxdemux/ChannelControlBlock.java
@@ -124,7 +124,6 @@ public class ChannelControlBlock implements 
IChannelControlBlock {
     }
 
     void reportRemoteError(int ecode) {
-        ri.flush();
         ri.getFullBufferAcceptor().error(ecode);
         remoteClose.set(true);
     }

Reply via email to