Todd Lipcon has submitted this change and it was merged. Change subject: Add a cached boolean whether DMS is empty ......................................................................
Add a cached boolean whether DMS is empty On servers which store a couple TB of data, I see a good portion of a core being burned in DeltaTracker::DeltaMemStoresEmpty(). Even the component lock is a rw-lock, the atomic instructions necessary to acquire the lock cost a lot. This adds a boolean member to DeltaTracker which is set on update to try to reduce this CPU load. Change-Id: I721ee74a112d968a33a1a82419b3268c08b67c79 Reviewed-on: http://gerrit.cloudera.org:8080/3004 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/tablet/delta_tracker.cc M src/kudu/tablet/delta_tracker.h 2 files changed, 15 insertions(+), 7 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3004 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I721ee74a112d968a33a1a82419b3268c08b67c79 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
