v2.12.1 NPE：在父子线程情况下`TransmittableThreadLocal<Map<String, String>>`新增map中的key值时

chen-shu-hao commented 2 years ago

在父子线程情况下TransmittableThreadLocal<Map<String, String>> holder新增map中的key值时，会导致原有key对应的value值一定概率丢失为null，出现空指针NPE。

oldratlee commented 2 years ago

TransmittableThreadLocal.holder 的类型是 InheritableThreadLocal<WeakHashMap<TransmittableThreadLocal<Object>, ?>>
和你描述的holder类型 ThreadLocal<Map<String, String>> 不同。

https://github.com/alibaba/transmittable-thread-local/blob/a588b590232ddd04033d316bc25791afea772392/src/main/java/com/alibaba/ttl/TransmittableThreadLocal.java#L199-L216

下面假设你说的holder不是指 TransmittableThreadLocal.holder，而是你自己业务里的context。

一般和使用方式相关，主要要注意的可能是

并发问题
数据的设置与传递，map跨线程传的引用，会被其它线程修改意料之外的修改。如果用的不是并发安全的map，还包含了并发Bug。
ThreadLocal初始化是lazy的。来源/父线程没有设置过值时（调用ThreadLocal.get()方法也会触发ThreadLocal.initValue()设置值），是不会传递的。子线程会通过ThreadLocal.initValue()获得值，如果没有重载过返回的是null。

这些方面的问题理解清楚了，应该是可以构造确定性复现的demo的。这样并发相关的问题基本上独立于 TTL，只要有并发就要注意并解决好。

下面假设你说的holder就是指 TransmittableThreadLocal.holder。

新增map中的key值会导致原有key对应的value值一定概率丢失为null,出现空指针

TransmittableThreadLocal.holder 这个map 设计实现成： value 只能是 null。

PS: TransmittableThreadLocal.holder 是TTL的最核心数据，因为有大量的生产环境的使用，有问题的可能性不大。需要了解结合你的使用方式，有更完整的信息，以排查。

无论哪种情况，@chen-shu-hao 给一下问题的信息：如NPE的栈、你的使用方式、环境等等

期望能从你这边出这个问题的业务代码中，请分离整理一个可以（尽量有较大概率）复现问题的极简可运行Demo工程：❤️

作为一个github工程，这样大家可以用来排查
就像这个Issue https://github.com/alibaba/transmittable-thread-local/issues/226#issuecomment-739314037 给的liauraljl/ttl-agent-demo这个工程

否则：

涉及业务代码与流程，不方便有效分析与排查
不能方便确定可能是哪个使用方式引发了这个问题无论是TTL的问题还是业务使用方式问题，都一样值得排查解决掉。 😁

chen-shu-hao commented 2 years ago

是第一种情况，holder是我们项目里的context，不是您说的TransmittableThreadLocal.holder。
确实我使用的是hashmap，没有使用线程安全的map。
在主线程中给holder中的map赋完值后，去子线程中给map put了新值。
- 这个值是我想往下传递的值，但是每个线程里生成的value不一致，所以我给一个固定的key值前面加了当前线程名称作为前缀，企图在接下来的流程中，每个线程取到自己的value值
- 然而在每个线程中put了多次这个key值后，map中原有的key值就出现了value为null的情况

现在我解决了这个问题：重新定义了一个新的TransmittableThreadLocal，没有使用主线程的TransmittableThreadLocal。

oldratlee commented 2 years ago

@chen-shu-hao 解决了问题就好。❤️

这个 Issue 先 Close 了。如果还有问题可以继续讨论。

liuyq913 commented 2 years ago

private static TransmittableThreadLocal<Map<String, String>> transmittableThreadLocal = new TransmittableThreadLocal();

子线程修改了transmittableThreadLocal里面某个值，会透传到主线程里面

oldratlee commented 2 years ago

private static TransmittableThreadLocal<Map<String, String>> transmittableThreadLocal = new TransmittableThreadLocal(); 子线程修改了transmittableThreadLocal里面某个值，会透传到主线程里面

是的。缺省传递的是引用。

如果业务有修改并且没恰当的同步，业务这样的通信方式是不线程安全的使用方式，即有 bug。已有不少 issue 讨论过，可以找了看一下。

通过重写TransmittableThreadLocal#copy方法，可以定制传递方式。参见项目的用户文档。 @liuyq913

alibaba / transmittable-thread-local

v2.12.1 NPE：在父子线程情况下`TransmittableThreadLocal<Map<String, String>>`新增map中的key值时 #333