最近在项目上碰到这样一个问题:
项目稳定了大半年,突然冒出一个
居然提示我无法创建新的线程了,当时突然意识到使用线程池的时候有问题,然后检测了一下代码,发现自己犯了个比较低级的错误:
@PostMapping("/refunds") @ResponseBody public void xxxx(){ ThreadFactory refundThreadFactory = new ThreadFactoryBuilder().setNameFormat("batchRefund").build(); ExecutorService executors = new ThreadPoolExecutor(20,50,0L, TimeUnit.MILLISECONDS, new LinkedBlockingQueue<Runnable>(),refundThreadFactory,new ThreadPoolExecutor.AbortPolicy()); executors.excute(xxxx); }
咋看之下没啥问题==不过没考虑到每次接口调用,都会创建新的线程池,而我的线程池又没有shutdown,导致每调用一次,都会在内存中创建一个线程池,这样越积越多,最后就会提示无法创建新的线程呢==
本次bug提醒了我线程池使用的时候虽然可以提高效率,但是也要及时关闭,可以把线程池创建移到方法外面,保证这个类的方法使用同一个线程,这样不关闭也行。或者在finally里面执行executors.shutdown();保证方法完成了会自动关闭