Adar Dembo has posted comments on this change.

Change subject: WIP: simplify MemTracker and move process throttling elsewhere
......................................................................


Patch Set 2:

> do you have any reservations about this particular patch?
 > Or just suggesting it might be easier to start clean?
 
No reservations, just thought it'd be net simpler to start clean.

 > If you aren't strongly against, I'd like to continue down this
 > route.

Sure, works for me. I'll review this in depth once you're comfortable that it's 
no longer WIP.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id16bad7d9a29a83e820a38e9d703811391cffe90
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to