Joe McDonnell has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9576
Change subject: IMPALA-6638: Reduce file handle cache lock contention ...................................................................... IMPALA-6638: Reduce file handle cache lock contention FileHandleCache::OpenFileHandle() currently holds the lock while opening a file handle. This lengthens the duration holding the lock considerably, causing contention when a lot of file handles are being opened (i.e. when the cache is cold). This changes FileHandleCache::OpenFileHandle() drops the lock while opening the file handle, then reacquires it to add the file handle to the cache. Performance tests confirm that this fixes contention when the file handle cache is cold. It does not change performance when the cache is hot. Change-Id: I4c695b21ca556e9c73c703c0c891e64939271c8d --- M be/src/runtime/io/handle-cache.inline.h 1 file changed, 28 insertions(+), 24 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/76/9576/1 -- To view, visit http://gerrit.cloudera.org:8080/9576 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I4c695b21ca556e9c73c703c0c891e64939271c8d Gerrit-Change-Number: 9576 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>