FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Properly close the Apache response so that connections can be reused by agherardi · Pull Request #3752 · javaee/jersey · GitHub

This repository was archived by the owner on May 28, 2018. It is now read-only.
  1. javaee
  2. jersey

Repository navigation

Properly close the Apache response so that connections can be reused - #3752

Open
agherardi wants to merge 1 commit into
javaee:masterfrom
agherardi:properlyClose
ConversationCommits (1)ChecksFiles changed
Open

Properly close the Apache response so that connections can be reused#3752
agherardi wants to merge 1 commit into
javaee:masterfrom
agherardi:properlyClose

Conversation

agherardi commented Jan 8, 2018

Copy link
Copy Markdown
Contributor

No description provided.

agherardi commented Jan 8, 2018

Copy link
Copy Markdown
Contributor Author

Obsoletes https://github.com/jersey/jersey/pull/3565

OCA http://www.oracle.com/technetwork/community/oca-486395.html

agherardi mentioned this pull request Jan 8, 2018
Closed
jansupol added the OCA label Jan 8, 2018

agherardi commented Jan 8, 2018

Copy link
Copy Markdown
Contributor Author

See http://hc.apache.org/httpcomponents-client-4.5.x/tutorial/html/fundamentals.html#d5e145

In order to ensure proper release of system resources one must close either the content stream associated with the entity or the response itself
The difference between closing the content stream and closing the response is that the former will attempt to keep the underlying connection alive by consuming the entity content while the latter immediately shuts down and discards the connection.

agherardi force-pushed the properlyClose branch 2 times, most recently from 4e59ad3 to 588b4e8 Compare January 10, 2018 17:36
agherardi mentioned this pull request Jan 10, 2018
Closed
jerseyrobot mentioned this pull request Apr 20, 2018
Closed
agherardi mentioned this pull request Jun 17, 2018
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

OCA

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

Back | FazBrowse Home | New Git URL