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

David Mollitor commented on TEZ-4275:
-------------------------------------

[~jeagles] This particular feature has been around since the dawn of time: 
Guava 3.0

https://guava.dev/releases/16.0.1/api/docs/com/google/common/collect/Interners.html

So there is less risk here of backwards compatibility. 

> Use Google Guava Intern Facility
> --------------------------------
>
>                 Key: TEZ-4275
>                 URL: https://issues.apache.org/jira/browse/TEZ-4275
>             Project: Apache Tez
>          Issue Type: Improvement
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Major
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Google Guava has a pretty cool Interner facility.
>  
>  * More memory efficient than current offering.  Map contains a weak key and 
> a static dummy value (current implementation uses a weak value)
>  * Current implementation has a single lock around the entire data structure. 
>  Google segments their data structure into (default: 4) segments for better 
> concurrency
>  * All the other thoughtful stuff Google has added into this feature



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to