hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-12475) Replace guava Cache with ConcurrentHashMap for caching Connection in ipc Client
Date Fri, 16 Oct 2015 01:55:09 GMT

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

Hudson commented on HADOOP-12475:
---------------------------------

FAILURE: Integrated in Hadoop-Yarn-trunk #1277 (See [https://builds.apache.org/job/Hadoop-Yarn-trunk/1277/])
HADOOP-12475. Move attribution to 2.8.0 section of CHANGES.txt. (cnauroth: rev 5f3f0e0f707ca94c62955e57c6b94c43a6321ea6)
* hadoop-common-project/hadoop-common/CHANGES.txt


> Replace guava Cache with ConcurrentHashMap for caching Connection in ipc Client
> -------------------------------------------------------------------------------
>
>                 Key: HADOOP-12475
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12475
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: conf, io, ipc
>            Reporter: Walter Su
>            Assignee: Walter Su
>             Fix For: 2.8.0
>
>         Attachments: HADOOP-12475.01.patch, HADOOP-12475.02.patch, HADOOP-12475.03.patch
>
>
> quote [~daryn] from HADOOP-11772:
> {quote}
> CacheBuilder is obscenely expensive for concurrent map, and it requires generating unnecessary
garbage even just to look up a key. Replace it with ConcurrentHashMap.
> I identified this issue that impaired my own perf testing under load. The slowdown isn't
just the sync. It's the expensive of Connection's ctor stalling other connections. The expensive
of ConnectionId#equals causes delays. Synch'ing on connections causes unfair contention unlike
a sync'ed method. Concurrency simply hides this.
> {quote}
> BTW, guava Cache is heavyweight. Per local test, ConcurrentHashMap has better overal
performance.



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

Mime
View raw message