<div dir="ltr">Is it safe to assume that the disableThreadLocals toggle will only be released along with a decent alternative (e.g., ScopedValues)? Otherwise, libraries relying on TLs will simply not work due to the thrown UnsupportedOperationException. Put another way, Log4j can replace TLs that serve object pooling purposes with something else, today; though for TLs that are used to pass context, disableThreadLocals=true will simply render them broken without a solution.<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 2, 2022 at 11:30 AM Ron Pressler <<a href="mailto:ron.pressler@oracle.com">ron.pressler@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">



<div style="overflow-wrap: break-word;">
<div style="margin:0px;font-stretch:normal;font-size:13px;line-height:normal;font-family:"Helvetica Neue"">
There is no difference in capabilities or behaviour between virtual threads and platform threads when it comes to ThreadLocals. It’s just that because some particular uses of TLs make an implicit assumption about threads being shared and few in number, assumptions
 that are usually true for platform threads and usually untrue for virtual threads, we’ve added the possibility of disabling ThreadLocals for any kind of thread. By default, all threads allow TLs, and so running old code on virtual threads will work as before.</div>
<div style="margin:0px;font-stretch:normal;font-size:13px;line-height:normal;font-family:"Helvetica Neue";min-height:15px">
<br>
</div>
<div style="margin:0px;font-stretch:normal;font-size:13px;line-height:normal;font-family:"Helvetica Neue"">
Just to be clear, there is nothing *fundamentally* wrong about TLs, nor their use in virtual threads. It’s just that thread usage patterns have led to *some* TL usage patterns that would be problematic with common virtual thread usage patterns. Many if not
 most uses of TLs â€” those that associate domain context with a task â€” are fine with virtual threads (although ScopedValues are much better). There is no need to deprecate ThreadLocal nor a plan to ever do so.</div>
<div style="margin:0px;font-stretch:normal;font-size:13px;line-height:normal;font-family:"Helvetica Neue"">
<br>
</div>
<div style="margin:0px;font-stretch:normal;font-size:13px;line-height:normal;font-family:"Helvetica Neue"">
— Ron</div>
<div><br>
<blockquote type="cite">
<div>On 2 Nov 2022, at 00:03, Carter Kozak <<a href="mailto:ckozak@ckozak.net" target="_blank">ckozak@ckozak.net</a>> wrote:</div>
<br>
<div>
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none">
<div>> That's right but not specific to virtual threads. It is also possible to <br>
</div>
<div>> create platform threads that do not support TLs. This is something that  <br>
</div>
<div>> we need a lot more feedback on before deciding if that part of the API <br>
</div>
<div>> should be made permanent.<br>
</div>
<div><br>
</div>
</div>
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none">
This is an interesting point. I'm not sure I understand why the introduction of virtual threads is the best place to make the ThreadLocal language feature configurable, but I'd appreciate additional context.<br>
</div>
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none">
The JEP describes the motivation for the dynamic configuration of ThreadLocals, as well as an alternative (jep-429 scoped values), however the behavior seems (to an outsider) to oppose the higher-level design of virtual threads which are meant to be otherwise
 a drop-in replacement for platform threads. I'd much rather fail predictably (or in a way the compiler can warn me about) rather than when a legacy function compiled with jdk1.5 executes on a particular type of thread.<br>
</div>
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none">
Given that using virtual threads is going to be a decision that happens in isolation from code that may use ThreadLocals, ideally we could introduce alternatives to ThreadLocal safely in an LTS before making ThreadLocal throw. Then, we could add the configuration
 point to disable ThreadLocals (on all types of thread?) and/or for potentially greater consistency, deprecate ThreadLocal for removal.<br>
</div>
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none">
<br>
</div>
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;text-decoration:none">
<div>
<div>
<div>> It would be interesting to know if log4j has done any recent evaluation<br>
</div>
</div>
<div>> with newer JDK releases and GCs to see their object pooling has a<br>
</div>
<div>
<div>> positive or negative benefit.<br>
</div>
<div><br>
</div>
</div>
<div>
<div>Yes, it's complicated. Performance has certainly changed over time with JDK releases and the benefits are use-case dependent. The primary benefit is isolation (lack of impact on GC, which itself may impact seemingly unrelated components).<br>
</div>
</div>
<div><br>
</div>
<div>Thanks,<br>
</div>
<div>Carter Kozak</div>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</div>

</blockquote></div>