[jira] [Commented] (THRIFT-5022) TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

2019-11-22 Thread Qinghui Xu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16980121#comment-16980121 ] Qinghui Xu commented on THRIFT-5022: And thanks for the fix > TIOStreamTransport.isOpen is false

[jira] [Commented] (THRIFT-5022) TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

2019-11-22 Thread Qinghui Xu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16980118#comment-16980118 ] Qinghui Xu commented on THRIFT-5022: Using a finally clause is definitly better for doing the

[jira] [Commented] (THRIFT-5022) TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

2019-11-21 Thread Andy Seaborne (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16979323#comment-16979323 ] Andy Seaborne commented on THRIFT-5022: --- This is bringing concerns about error handling – great to

[jira] [Commented] (THRIFT-5022) TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

2019-11-20 Thread Qinghui Xu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16978780#comment-16978780 ] Qinghui Xu commented on THRIFT-5022: Yes, My concern is that there might be some uncaught errors

[jira] [Commented] (THRIFT-5022) TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

2019-11-20 Thread Andy Seaborne (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16978737#comment-16978737 ] Andy Seaborne commented on THRIFT-5022: --- [~q.xu] Thank you for the quick response. I've just done

[jira] [Commented] (THRIFT-5022) TIOStreamTransport.isOpen is false for InputStream or Outpstream only use.

2019-11-20 Thread Qinghui Xu (Jira)
[ https://issues.apache.org/jira/browse/THRIFT-5022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16978708#comment-16978708 ] Qinghui Xu commented on THRIFT-5022: When I fixed THRIFT-2530, I did not notice such use cases of