You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@thrift.apache.org by "Roger Meier (Resolved) (JIRA)" <ji...@apache.org> on 2012/02/19 19:54:40 UTC

[jira] [Resolved] (THRIFT-1517) TTransport.ReadAll() should set exception type to EndOfFile

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

Roger Meier resolved THRIFT-1517.
---------------------------------

    Resolution: Fixed

Thanks, committed.
                
> TTransport.ReadAll() should set exception type to EndOfFile
> -----------------------------------------------------------
>
>                 Key: THRIFT-1517
>                 URL: https://issues.apache.org/jira/browse/THRIFT-1517
>             Project: Thrift
>          Issue Type: Bug
>          Components: C# - Library
>    Affects Versions: 0.8
>            Reporter: Stefan Gmeiner
>            Priority: Minor
>         Attachments: THRIFT-1517.patch
>
>
> The TTransport.ReadAll() throws a TTransportException(string) if read returns <= 0.  The exception text is "Cannot read, Remote side has closed." but the exception type is not set.  To allow checking for EOF condition in catch blocks I suggest to set the type to TTransportException.ExceptionType.EndOfFile.  BTW this is the same as in the C++ library.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira