[
https://issues.apache.org/jira/browse/TEZ-3707?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Harish Jaiprakash updated TEZ-3707:
-----------------------------------
Attachment: TEZ-3707.01.patch
Fixed isTerminated to check if pending futures are already done. No extra
tests, tested manually by putting sleep in the done method and verifying that
isTerminated returns true.
> TezSharedExecutor race condition in awaitTermination vs isTerminated.
> ---------------------------------------------------------------------
>
> Key: TEZ-3707
> URL: https://issues.apache.org/jira/browse/TEZ-3707
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Harish Jaiprakash
> Attachments: TEZ-3707.01.patch
>
>
> awaitTermination finishes if all the futures are resolved. isTerminated
> returns true if futures is empty. future done is invoked after the future is
> resolve, which can lead to awaitTermination finishing but isTerminated is
> false until done is invoked.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
