[ 
https://issues.apache.org/jira/browse/PIG-4295?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14517269#comment-14517269
 ] 

Xuefu Zhang commented on PIG-4295:
----------------------------------

I suspect that PIG-1784 might have made the cheap fix in PigContext by changing 
the static variable to a thread-local variable. On the surface, PigServer 
already has a reference to PigContext, so in theory each PigServer should has 
its own PigContext instance. The original problem was that PigServer has static 
variables. Thus, the fix should have been making them non-static. Of course 
doing this could be some other complications, but making them thread-local 
seemed to be a lazy fix for the problem.

[~alangates], [~rding], any insights on this?

I'd suggest investigating the original concurrent issue and proposing an 
alternative fix. If so, we need to create a JIRA on trunk.

> Enable unit test "TestPigContext" for spark
> -------------------------------------------
>
>                 Key: PIG-4295
>                 URL: https://issues.apache.org/jira/browse/PIG-4295
>             Project: Pig
>          Issue Type: Sub-task
>          Components: spark
>            Reporter: liyunzhang_intel
>            Assignee: liyunzhang_intel
>             Fix For: spark-branch
>
>         Attachments: PIG-4295.patch, 
> TEST-org.apache.pig.test.TestPigContext.txt
>
>
> error log is attached



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to