Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21956 )
Change subject: KUDU-613: Fix minor metrics related bug ...................................................................... KUDU-613: Fix minor metrics related bug When upserting in the protected segment, the state of the handle isn't updated properly (the variable in_protected_segment). This variable is used later when releasing the handle to determine which segment's metrics to update. This patch fixes this bug and adds appropriate test coverage to verify the right metrics are being updated. Change-Id: I0d2fd44169e1abe642c11e16e4b87aaa971b2d88 Reviewed-on: http://gerrit.cloudera.org:8080/21956 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/util/slru_cache-test.cc M src/kudu/util/slru_cache.cc M src/kudu/util/slru_cache.h 3 files changed, 55 insertions(+), 1 deletion(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21956 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0d2fd44169e1abe642c11e16e4b87aaa971b2d88 Gerrit-Change-Number: 21956 Gerrit-PatchSet: 6 Gerrit-Owner: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]>
