> These changes make sure that pending requests are terminated if the selector 
> manager thread exits due to exceptions.
> This includes:
>    1. completing CompletableFutures that were returned to the caller code
>    2. cancelling requests that are in flight
>    3. calling onError on BodySubscribers that may not have been completed
> Note that step 3 is necessary as certain CompletableFutures, such as those 
> returned by `BodySubscribers.ofInputStream`, complete immediately, the 
> operation being eventually completed when the last bite of the response is 
> read. Completing a completable future that is already completed has no 
> effect, this case is handled by completing the BodySubscriber too.

Daniel Fuchs has updated the pull request incrementally with two additional 
commits since the last revision:

 - Fixed exception handling in tests
 - Better exception reporting

-------------

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/7196/files
  - new: https://git.openjdk.java.net/jdk/pull/7196/files/c86e3766..4953167a

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7196&range=05
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7196&range=04-05

  Stats: 202 lines in 6 files changed: 82 ins; 90 del; 30 mod
  Patch: https://git.openjdk.java.net/jdk/pull/7196.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/7196/head:pull/7196

PR: https://git.openjdk.java.net/jdk/pull/7196

Reply via email to