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

Siddharth Seth commented on TEZ-2082:
-------------------------------------

Oops. Sorry about that. Already have this fixed in follow up changes intended 
for TEZ-2003. Didn't realize it existed in the main branch as well.

The race is expected, and can't really be controlled. It existed with the old 
code as well - but the null bit was handled better there.

The patch looks good. Not sure if we need the pre-condition though. The trace 
should help debug this scenario.

> Failing test: TestPreemption::testPreemptionWithSession/
> --------------------------------------------------------
>
>                 Key: TEZ-2082
>                 URL: https://issues.apache.org/jira/browse/TEZ-2082
>             Project: Apache Tez
>          Issue Type: Bug
>    Affects Versions: 0.7.0
>            Reporter: Hitesh Shah
>            Assignee: Bikas Saha
>         Attachments: TEZ-2082.1.patch
>
>
> From 
> https://builds.apache.org/job/Tez-Build/891/testReport/junit/org.apache.tez.dag.app/TestPreemption/testPreemptionWithSession/
> Exception in thread "Thread-27" java.lang.NullPointerException
>       at 
> org.apache.tez.dag.app.TaskAttemptListenerImpTezDag.getTask(TaskAttemptListenerImpTezDag.java:222)
>       at 
> org.apache.tez.dag.app.MockDAGAppMaster$MockContainerLauncher.run(MockDAGAppMaster.java:230)
>       at java.lang.Thread.run(Thread.java:662)



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

Reply via email to