Todd Lipcon has uploaded a new change for review. http://gerrit.cloudera.org:8080/2707
Change subject: IMPALA-3291. Pause monitor thread should check tcmalloc performance ...................................................................... IMPALA-3291. Pause monitor thread should check tcmalloc performance IMPALA-3114 added a "pause monitor" thread which wakes up periodically to check that the process has not paused. However, it doesn't do anything when it wakes up aside from check that it didn't sleep longer than expected. Since it does no allocation, it won't notice pauses like the one described in IMPALA-2800 where tcmalloc is holding a process-wide lock for many seconds. This patch makes the pause monitor do an allocation and free pair, and ensures that the allocation goes through the tcmalloc central free list rather than a thread cache. This ensures that the heap lock is not blocked for an undue amount of time. Change-Id: I40516456dda9ea6cfce3f2074ab9a4b25087c4bd --- M be/src/common/init.cc 1 file changed, 20 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/07/2707/1 -- To view, visit http://gerrit.cloudera.org:8080/2707 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I40516456dda9ea6cfce3f2074ab9a4b25087c4bd Gerrit-PatchSet: 1 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Todd Lipcon <[email protected]>
