Github user prashantv commented on the pull request:
https://github.com/apache/thrift/pull/517#issuecomment-111205380
@i Just started a PR for the same issue, didn't see your fix.
I think it's better to not set p.Response, since it affects other code
(IsOpen returns true, Read will allow reading bytes which most likely won't
match the protocol, etc). I think it might be better to just close the Body on
an error: #518
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
