{"action":"created","issue":{"url":"https://api.github.com/repos/apache/jena/issues/151","repository_url":"https://api.github.com/repos/apache/jena","labels_url":"https://api.github.com/repos/apache/jena/issues/151/labels{/name}","comments_url":"https://api.github.com/repos/apache/jena/issues/151/comments","events_url":"https://api.github.com/repos/apache/jena/issues/151/events","html_url":"https://github.com/apache/jena/pull/151","id":164170653,"number":151,"title":"JENA-576: Moving away from deprecated HttpCommons Client API","user":{"login":"ajs6f","id":110994,"avatar_url":"https://avatars.githubusercontent.com/u/110994?v=3","gravatar_id":"","url":"https://api.github.com/users/ajs6f","html_url":"https://github.com/ajs6f","followers_url":"https://api.github.com/users/ajs6f/followers","following_url":"https://api.github.com/users/ajs6f/following{/other_user}","gists_url":"https://api.github.com/users/ajs6f/gists{/gist_id}","starred_url":"https://api.github.com/users/ajs6f/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajs6f/subscriptions","organizations_url":"https://api.github.com/users/ajs6f/orgs","repos_url":"https://api.github.com/users/ajs6f/repos","events_url":"https://api.github.com/users/ajs6f/events{/privacy}","received_events_url":"https://api.github.com/users/ajs6f/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":92,"created_at":"2016-07-06T20:50:12Z","updated_at":"2016-10-14T17:47:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/jena/pulls/151","html_url":"https://github.com/apache/jena/pull/151","diff_url":"https://github.com/apache/jena/pull/151.diff","patch_url":"https://github.com/apache/jena/pull/151.patch"},"body":"This is a bit of a shake-up in the HTTP machinery, but as per discussion with @afs , a \"big bang\" discarding the use of mutable HTTP clients seems like the least painful approach to move forward. Several pieces of Jena machinery are removed in this, including `HttpAuthenticator` and its impls, in favor of directly using HTTP Commons machinery to the same ends. This will cause some annoyance to folks who have used those Jena types, but I think we can reasonably mitigate that with good release docs and readiness to help on-list.\r\n\r\nThere is at least one TODO that remains here, which is to reenable the total-read-timeout for `QueryEngineHTTP`. The immutable nature of HTTP clients in the current HTTP Commons API means that it's no longer a simple matter to just pass through that timeout to HTTP machinery. I will work up a [replacement piece](https://stackoverflow.com/questions/6764035/apache-httpclient-timeout) using `ScheduledThreadPoolExecutor` to timeout running requests, but I wanted to get this out there and let folks react to it to make sure that I'm on the right road before going any further."},"comment":{"url":"https://api.github.com/repos/apache/jena/issues/comments/253871985","html_url":"https://github.com/apache/jena/pull/151#issuecomment-253871985","issue_url":"https://api.github.com/repos/apache/jena/issues/151","id":253871985,"user":{"login":"afs","id":101362,"avatar_url":"https://avatars.githubusercontent.com/u/101362?v=3","gravatar_id":"","url":"https://api.github.com/users/afs","html_url":"https://github.com/afs","followers_url":"https://api.github.com/users/afs/followers","following_url":"https://api.github.com/users/afs/following{/other_user}","gists_url":"https://api.github.com/users/afs/gists{/gist_id}","starred_url":"https://api.github.com/users/afs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afs/subscriptions","organizations_url":"https://api.github.com/users/afs/orgs","repos_url":"https://api.github.com/users/afs/repos","events_url":"https://api.github.com/users/afs/events{/privacy}","received_events_url":"https://api.github.com/users/afs/received_events","type":"User","site_admin":false},"created_at":"2016-10-14T17:47:48Z","updated_at":"2016-10-14T17:47:48Z","body":"I don't know what state the PR is in.  I don't see the changes I made today to sort things out on the PR so I am unclear what \"it\" is and what your happy comment refers to.\r\n\r\nI can merge it,or you can, as-is together with http_op_3 commit 944e520 and then resolve JENA-576 so it is recorded for this release. \r\n\r\n"}}
{"actor":{"display_login":"afs"}}