You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (Commented) (JIRA)" <ji...@apache.org> on 2012/02/19 20:44:34 UTC
[jira] [Commented] (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:comment-tabpanel&focusedCommentId=13211510#comment-13211510 ]
Hudson commented on THRIFT-1517:
--------------------------------
Integrated in Thrift #407 (See [https://builds.apache.org/job/Thrift/407/])
THRIFT-1517 TTransport.ReadAll() should set exception type to EndOfFile
Patch: Stefan Gmeiner (Revision 1291039)
Result = SUCCESS
roger : http://svn.apache.org/viewvc/?view=rev&rev=1291039
Files :
* /thrift/trunk/lib/csharp/src/Transport/TTransport.cs
> 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