[ https://issues.apache.org/jira/browse/PIG-3844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13951182#comment-13951182 ]
Akihiro Matsukawa commented on PIG-3844: ---------------------------------------- We have a PPNL that extracts information such as the alias and feature from the ScriptState that runs on a separate thread from the main pig thread, which requires that it inherits a copy of the ThreadLocal object when it is spun off. I will add a test and generate a new patch this weekend. Thanks! > Make ScriptState InheritableThreadLocal for threads that need it > ---------------------------------------------------------------- > > Key: PIG-3844 > URL: https://issues.apache.org/jira/browse/PIG-3844 > Project: Pig > Issue Type: Improvement > Reporter: Akihiro Matsukawa > Assignee: Akihiro Matsukawa > Attachments: PIG-3844.patch > > > I have a PPNL that is forked off the main thread so that its operations does > not block pig from continuing to run. This PPNL needs the ScriptState, but is > not able to get it in pig13 because the ScriptState is ThreadLocal. > In pig12, this worked because there was logic to start a new ScriptState on > ScriptState.get() which was removed in PIG-3525 > (https://reviews.apache.org/r/15634). > My proposal is to change ScriptState to be InheritableThreadLocal, so that > any new child threads that are spawned will have a copy of it. I have a > pretty limited understanding of pig, but I do not see anything that this > proposed change would break. -- This message was sent by Atlassian JIRA (v6.2#6252)