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

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.8
• Component/s: C# - Library
• Patch Info: Patch Available
• Assignee: Unassigned
• Reporter: Stefan Gmeiner
• Created: 17/Feb/12 07:10
• Updated: 02/Mar/12 04:57
• Resolved: 19/Feb/12 18:53

Description

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.