Github user ajs6f commented on the issue:

    https://github.com/apache/jena/pull/151
  
    Reading through the current HTTP Commons docs leaves me with the impression 
that we do not need to close clients after use in `HttpOp`, but that we do need 
to see to it that _responses_ are closed. However, tracing through to 
`oaj.riot.web.HttpResponseLib.nullResponse` in conjunction with advice 
[here](https://hc.apache.org/httpcomponents-client-ga/tutorial/html/fundamentals.html#d5e145)
 assures me that we are doing what we need to do. The only case that I find 
uncertain is in methods like `TypedInputStream execHttpGet(String url)` wherein 
we really cannot be sure that the request is properly closed. I'm not sure what 
to do about this short of trying to take `HttpOp` completely internal, which 
seems like a massive overreaction.


---
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.
---

Reply via email to