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

Posted to dev@thrift.apache.org by jeking3 <gi...@git.apache.org> on 2017/09/05 04:56:47 UTC

[GitHub] thrift pull request #1335: Add default message for TApplicationException

Github user jeking3 commented on a diff in the pull request:

    https://github.com/apache/thrift/pull/1335#discussion_r136895273
  
    --- Diff: lib/go/thrift/application_exception.go ---
    @@ -30,6 +30,17 @@ const (
     	PROTOCOL_ERROR                 = 7
     )
     
    +var defaultApplicationExceptionMessage = map[int32]string{
    +	UNKNOWN_APPLICATION_EXCEPTION:  "unknown application exception",
    +	UNKNOWN_METHOD:                 "unknown method",
    +	INVALID_MESSAGE_TYPE_EXCEPTION: "invalid message type",
    +	WRONG_METHOD_NAME:              "wrong method name",
    +	BAD_SEQUENCE_ID:                "bad sequnce ID",
    --- End diff --
    
    Typo in the string


---