Tim Armstrong has posted comments on this change.

Change subject: IMPALA-3142: avoid spinlock in MemTracker::GcMemory()
......................................................................


Patch Set 1:

I sanity tested this locally on a 1-impalad minicluster by launching this query 
200 times simultaneously and checking that it behaved as expected (memory limit 
increased until the process limit, then a bunch of queries fail).

  use tpch_20_parquet;
  select distinct * from lineitem limit 5;

-- 
To view, visit http://gerrit.cloudera.org:8080/2459
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia9e91ea67ea924ba9b81e7376487838b365ca8ce
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.5.0_5.7.0
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to