Merge pull request #213 from bsinno/fix_avoid_loosing_data_if_pool_maxed_out
Set new policy for central executor pool to avoid loosing data.
This commit is contained in:
@@ -13,6 +13,7 @@ import java.util.concurrent.BlockingQueue;
|
||||
import java.util.concurrent.Executor;
|
||||
import java.util.concurrent.ScheduledExecutorService;
|
||||
import java.util.concurrent.ThreadPoolExecutor;
|
||||
import java.util.concurrent.ThreadPoolExecutor.CallerRunsPolicy;
|
||||
import java.util.concurrent.TimeUnit;
|
||||
|
||||
import org.slf4j.Logger;
|
||||
@@ -60,15 +61,21 @@ public class ExecutorAutoConfiguration {
|
||||
public ThreadPoolExecutor threadPoolExecutor() {
|
||||
final BlockingQueue<Runnable> blockingQueue = new ArrayBlockingQueue<>(
|
||||
asyncConfigurerProperties.getQueuesize());
|
||||
final ThreadPoolExecutor threadPoolExecutor = new ThreadPoolExecutor(asyncConfigurerProperties.getCorethreads(),
|
||||
return new ThreadPoolExecutor(asyncConfigurerProperties.getCorethreads(),
|
||||
asyncConfigurerProperties.getMaxthreads(), asyncConfigurerProperties.getIdletimeout(),
|
||||
TimeUnit.MILLISECONDS, blockingQueue,
|
||||
new ThreadFactoryBuilder().setNameFormat("central-executor-pool-%d").build());
|
||||
threadPoolExecutor.setRejectedExecutionHandler((r, executor) -> LOGGER.warn(
|
||||
"Reject runnable for centralExecutorService, reached limit of queue size {}",
|
||||
executor.getQueue().size()));
|
||||
new ThreadFactoryBuilder().setNameFormat("central-executor-pool-%d").build(),
|
||||
new PoolSizeExceededPolicy());
|
||||
}
|
||||
|
||||
return threadPoolExecutor;
|
||||
private static class PoolSizeExceededPolicy extends CallerRunsPolicy {
|
||||
@Override
|
||||
public void rejectedExecution(final Runnable r, final ThreadPoolExecutor executor) {
|
||||
LOGGER.warn(
|
||||
"Caller has to run on its own instead of centralExecutorService, reached limit of queue size {}",
|
||||
executor.getQueue().size());
|
||||
super.rejectedExecution(r, executor);
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
|
||||
Reference in New Issue
Block a user