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

Galen O'Sullivan edited comment on GEODE-3170 at 7/12/17 7:06 PM:
------------------------------------------------------------------

This issue is caused by a socket that's closed on the client side, which causes 
a SocketException to be thrown in ProtobufProtocolSerializer.deserialize . This 
throws a InvalidProtocolMessageException wrapping the SocketException (as an 
IOException), which is then rethrown as an IOException (whew!).

We need to implement shutdown properly to have the shutdown method to call when 
the socket is no longer readable (hence the dependency on GEODE-3079).

I think it's reasonable to close a socket if we ever get an IOException from 
it. Are there other cases of IOException I'm missing?


was (Author: gosullivan):
This issue is caused by a socket that's closed on the client side, which causes 
a SocketException to be thrown in ProtobufProtocolSerializer.deserialize . This 
throws a InvalidProtocolMessageException wrapping the SocketException (as an 
IOException), which is then rethrown as an IOException (whew!).

We need to implement shutdown properly to have the shutdown method to call when 
the socket is no longer readable.

I think it's reasonable to close a socket if we ever get an IOException from 
it. Are there other cases of IOException I'm missing?

> Geode keeps logging 
> org.apache.geode.protocol.exception.InvalidProtocolMessageException
> ---------------------------------------------------------------------------------------
>
>                 Key: GEODE-3170
>                 URL: https://issues.apache.org/jira/browse/GEODE-3170
>             Project: Geode
>          Issue Type: Bug
>          Components: client/server
>            Reporter: Alexander Murmann
>            Assignee: Galen O'Sullivan
>             Fix For: 1.3.0
>
>         Attachments: Proto-parse-error (1).txt
>
>
> We've been seeing the logs getting filled up with exceptions if the new 
> binary protocol is enabled. This quickly fills up the hard drive.
> See attached file for example log



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

Reply via email to