问题排查步骤:
1. jstack查看jvm线程信息,发现已经有1w+线程存在
2. 看命名规范:pool-11238-thread-1大量存在,且处于waiting状态,由此可以猜测代码中存在一种ThreadPoolExecutor调用,coresize设置为1的地方每次调用完没有回收
3. 登录服务器,jmap -dump 文件,下载至本地
4. 使用jvisualvm工具查看dump文件,仔细查看pool-xxxxx-thread-1这类线程的ThreadPoolExecutor的实例信息,符合预测:
这完全吻合Executors.newFixedThreadPool(1)的创建方式
5. 代码中寻找使用了ThreadPoolExecutor或者Executors.newFixedThreadPool(1)的地方,找到使用的地方
6. 分析原因:
private ExecutorService executorService = Executors.newFixedThreadPool(1);
使用的时候期望是所有的实例公用一个,但是由于使用的guice,默认实例都是非单例的,所有每个类都会创建一份,这就导致两个问题:
a. 每次请求都会有一个新的实例,每个实例持有一个线程池
b. 由于线程池没有手动关闭是永远存在的,因此线程池一直会占用内存,并且占用栈,继续递增下去甚至会出现OOM
c. 线程越多,响应速度就会越慢
7. 如何解决
配置为private static final