[
https://issues.apache.org/jira/browse/TEZ-4090?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16947240#comment-16947240
]
Rajesh Balamohan commented on TEZ-4090:
---------------------------------------
Looks like non-sync WeakHashMap has chances of running into infinite loop.
https://bugs.openjdk.java.net/browse/JDK-8075006
So the better choice would be to make it synchronizedMap and remove the sync
block on method.
> TezIDCache::getInstance sync lock
> ---------------------------------
>
> Key: TEZ-4090
> URL: https://issues.apache.org/jira/browse/TEZ-4090
> Project: Apache Tez
> Issue Type: Bug
> Reporter: Rajesh Balamohan
> Priority: Minor
> Attachments: tez-task-id-concurrency.png
>
>
> profiler output shared by [~gopalv].
> Looking at the code, it is not mandatory to have sync on getInstance.
> https://github.com/apache/tez/blob/master/tez-common/src/main/java/org/apache/tez/dag/records/TezID.java#L48
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
