ThreadLoacl的内存泄露问题

线上有问题的时候，虽然每行日志都打印了流水号，但是请求存在重试的情况，这样想筛选出单一请求的日志比较麻烦，于是想使用SLF4j的MDC，MDC实际上就是一个ThreadLoacl，但是同事实现的日志trace中，没有进行MDC.remove(),我认为这样会存在内存泄漏，但是同事说，我每次对同一个key进行set和get，不需要remove吧，我当时一愣，竟说不出哪里不对，但是就hi感觉不对，于是重新看了一下Thread和ThreadLoacl、ThreadLoaclMap.

 

首先Thread内部有一个ThreadLoaclMap属性，这个map是以ThreadLoacl的实例为key（不是很准确，包装成了Entry，但是Entry是继承了WeakReference）,并不是以你ThreadLoacl.set的key为key，因为是WeakReference，所以gc会回收掉ThreadLoacl实例，但是ThreadLoacl的value实际上是在Thread内部组合的ThreadLoaclMap的value中的，这是一个强引用，如果线程不销毁，那么这个线程和value之间永远存在一个强引用，这个value的内存永远不会被gc，迟早有一天会oom。

这就让我想起了ThreadPoolExecutor的keepAliveTime，之前设置的时候对这个参数的理解不深，总觉得越大越好，这样省的超过core数目时new线程了，但是现在想一下，这个还是不要太大，否则，如果线程使用过程中有很多ThreadLoacl，那么会占用太多内存，应合理设置keepAliveTime这个参数来释放内存