You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Dan Hecht (Code Review)" <ge...@cloudera.org> on 2016/03/09 18:06:09 UTC

[Impala-CR](cdh5-trunk) IMPALA-3161: gperfutil SpinLock suffers from potential wakeup delay

Dan Hecht has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2500

Change subject: IMPALA-3161: gperfutil SpinLock suffers from potential wakeup delay
......................................................................

IMPALA-3161: gperfutil SpinLock suffers from potential wakeup delay

Backport this patch from gperfutil github:

>From 560ca8650c8a9d2971420970f0ed5e17848150eb Mon Sep 17 00:00:00 2001
From: "chappedm@gmail.com"
 <ch...@6b5cf1ce-ec42-a296-1ba9-69fdba395a50>
Date: Sun, 10 Mar 2013 20:02:46 +0000
Subject: [PATCH] issue-491: Significant performance improvement for spin lock
 contention

This patch fixes issues where spinlocks under contention were failing to
wakeup waiters, sometimes resulting in blow ups from 13ns to as high as 256ms.
Under heavy contention, applications were observed sleeping for minutes at a
time giving the appearance of a hang.

Change-Id: I660908144ee4525bde5511ca5252c9fba33d1cb2
---
M thirdparty/gperftools-2.0/src/base/spinlock.h
1 file changed, 2 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/00/2500/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2500
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I660908144ee4525bde5511ca5252c9fba33d1cb2
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Dan Hecht <dh...@cloudera.com>