avaje-http-client icon indicating copy to clipboard operation
avaje-http-client copied to clipboard

Published 20 hours ago • verified publisher icon avaje

Change such that Executor is used by default with async whenComplete

Open rbygrave opened this issue 2 years ago • 0 comments

Refer

  • https://bugs.openjdk.java.net/browse/JDK-8204339 - HTTP Client Dependent tasks should run in the common pool
  • https://bugs.openjdk.java.net/browse/JDK-8204679 - HTTP Client refresh
  • https://bugs.openjdk.java.net/browse/JDK-8209943 - HttpClient.Builder::executor minor spec correction for dependent tasks
  • https://stackoverflow.com/questions/51907641/java-11-http-client-asynchronous-execution

This PR changes avaje-http-client such that when an Executor is provided then async will by default use that given Executor rather than the ForkJoinPool.commonPool().

At this stage I'm not 100% sure if we should merge this change.

rbygrave avatar Dec 06 '21 02:12 rbygrave