JERSEY-2852: ClientResponse does not close ClosableHttpResponse by marcuslinke · Pull Request #164 · javaee/jersey · GitHub
Skip to content
This repository was archived by the owner on May 28, 2018. It is now read-only.

JERSEY-2852: ClientResponse does not close ClosableHttpResponse#164

Closed
marcuslinke wants to merge 3 commits intojavaee:masterfrom
docker-java:master
Closed

JERSEY-2852: ClientResponse does not close ClosableHttpResponse#164
marcuslinke wants to merge 3 commits intojavaee:masterfrom
docker-java:master

Conversation

@marcuslinke
Copy link
Copy Markdown

This fixes https://java.net/jira/browse/JERSEY-2852

Please review and merge. Thanks!

@mgajdos
Copy link
Copy Markdown
Contributor

mgajdos commented May 15, 2015

@AdamLindenthal
Copy link
Copy Markdown
Member

Hi Marcus,
have you noticed Michal's reponse? Any follow-up? I mean - are you still planning to continue the work on this?

Thanks,
Adam

@jerseyrobot
Copy link
Copy Markdown
Contributor

Can one of the admins verify this patch?

@AdamLindenthal
Copy link
Copy Markdown
Member

Jenkins, please test this patch.

@AdamLindenthal
Copy link
Copy Markdown
Member

Hi Marcus, will you sign and send the OCA? Or have you already done so?
Please, let us know. Also, we will need you to create a unit test for your change.

Thanks,
Adam

@AdamLindenthal
Copy link
Copy Markdown
Member

Hi Marcus,

I am sorry, but since you didn't respond since may and didn't sign the OCA, I am closing the PR.
Once you have more time to finish this, feel free to re-try.

Regards,
Adam

@KostyaSha
Copy link
Copy Markdown

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants