[ https://issues.apache.org/jira/browse/PIG-3844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13950082#comment-13950082 ]
Akihiro Matsukawa commented on PIG-3844: ---------------------------------------- Sure, no problem. I am also running "ant test" as we speak, I'm not quite sure exactly what subtest of tests that runs. > 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)