[GitHub] [kyuubi] bowenliang123 commented on pull request #5173: Explicitly close underlying httpclient when closing KyuubiConnection in HTTP transport mode

["bowenliang123 (via GitHub)" <gi...@apache.org>]

bowenliang123 commented on PR #5173:
URL: https://github.com/apache/kyuubi/pull/5173#issuecomment-1680723855

   Source code of libthrift 0.9.3  viewed on local IDEA: (Closing the input stream but not the connection)
   <img width="736" alt="image" src="https://github.com/apache/kyuubi/assets/1935105/f5236588-de28-4ccc-930b-f8fce5d504dd">
   
   Wile  viewed on Github (Closing boththe input stream and the connection)
   https://github.com/apache/thrift/blob/0.9.3/lib/javame/src/org/apache/thrift/transport/THttpClient.java
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org