Github user kdubb commented on the pull request:
https://github.com/apache/thrift/pull/539#issuecomment-146952528
I've brought the exception reporting in line with other languages. I
looked into Java & C++ to see the error codes they for all. Historically
TApplicationError seems to have been correct in the past and I trampled on it
by changing the codes themselves. The protocol & transport errors needed a bit
of shuffling but now they are using a meaningful code in the correct context.
I've kept some of the extended error information we were traditionally
reporting but put it into variables that do not escape the current environment.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
