[
https://issues.apache.org/jira/browse/PIG-3844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13950075#comment-13950075
]
Cheolsoo Park commented on PIG-3844:
------------------------------------
Hi Akihiro, thank you for the patch. PIG-3525 was my change, and sorry for any
convenience.
What you propose sounds good to me. Just to be safe, do you mind if I run full
unit tests? I can run it overnight.
> 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)
