Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8082 to look at the new patch set (#4). Change subject: Fix tablet state metrics race ...................................................................... Fix tablet state metrics race 6fef9ca5c9d1dcf571795c514cb9fecc493d0714 added tablet state summary metrics by adding a struct with cached counts that was periodically refreshed by walking the tablet manager, but also added a race between reading the cached counts and refreshing them. This fixes the race by centralizing reading and writing the counts in the tablet manager and protecting them with the tablet manager's lock. Change-Id: I88585c04692a6dc168b3750be9888769240cf3f6 --- M src/kudu/tserver/CMakeLists.txt M src/kudu/tserver/ts_tablet_manager.cc M src/kudu/tserver/ts_tablet_manager.h D src/kudu/tserver/ts_tablet_manager_metrics.cc D src/kudu/tserver/ts_tablet_manager_metrics.h 5 files changed, 106 insertions(+), 273 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/8082/4 -- To view, visit http://gerrit.cloudera.org:8080/8082 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I88585c04692a6dc168b3750be9888769240cf3f6 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>