Todd Lipcon has posted comments on this change.

Change subject: Start a background thread to run ResultTracker GC
......................................................................


Patch Set 2:

(1 comment)

will try to write a test. fwiw I've been testing this on a cluster ~2 days now 
and seems stable. Across ~72 tablet servers none is showing a peak memory 
consumption of more than 5MB (and averaging around 2MB).

http://gerrit.cloudera.org:8080/#/c/3961/2//COMMIT_MSG
Commit Message:

Line 7: Start a background thread to run ResultTracker GC
> I was expecting ResultTracker GC to be implemented as an MM op, like other 
yep, it's assumed that it's cheap enough (and doesnt do IO) so it doesn't need 
to be "prioritized" against other actions that do IO and thus might contend 
with each other.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia34ce95e78920596eb8b9db53643845f637c8e6c
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: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to