amplitude / Amplitude-Java

Official Amplitude Java SDK
MIT License
6 stars 10 forks source link

Amplitude client using ForkJoinPool for I/O #108

Open kevink-sq opened 2 weeks ago

kevink-sq commented 2 weeks ago

Expected Behavior

sendEvents should use sendThreadPool instead of the default ForkJoinPool.

Current Behavior

sendEvents uses the default ForkJoinPool via CompletableFuture.supplyAsync(runnable) which scales with the number of cpu.

Possible Solution

See https://github.com/amplitude/Amplitude-Java/pull/107

kevink-sq commented 2 weeks ago

cc: @izaaz