Every call to AttemptTimeLimiter.fixedTimeLimit(long, TimeUnit) creates a new CachedThreadPool

[rhuffman]

The TimeLimiters use an ExecutorService to which their Callables are submitted. A Future is retained and, when the time limit is reached, the Future is cancelled.

Currently, the AttemptTimeLimiters.fixedTimeLimit method that does not take an ExecutorService creates a new cached thread pool every time it is used. (See the FixedAttemptTimeLimiter constructor.)

Because each call creates a new cached thread pool, none of the threads from any of the thread pools will ever be reused. Each pool will create a single thread, that thread will be used, the thread will remain alive but inactive for sixty seconds then die. At that point the cached thread pool can be garbage collected.

Instead of create a new thread pool each time it is invoked that FixedAttemptTimeLimiter constructor should use the same instance of a cached thread pool every time.