You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Alexey Serbin (Code Review)" <ge...@cloudera.org> on 2019/04/06 06:43:10 UTC

[kudu-CR] [HaClient] introduce client metrics

Hello Tidy Bot, Andrew Wong, Hao Hao, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12951

to look at the new patch set (#2).

Change subject: [HaClient] introduce client metrics
......................................................................

[HaClient] introduce client metrics

Introduced metrics for HA Sentry client in SentryAuthzProvider.
Added corresponding test to verify how the newly introduced
metrics work.

Change-Id: I86e12ee6ee34f39c11f18b21e846cc5a1dee6b6f
---
M src/kudu/master/CMakeLists.txt
M src/kudu/master/catalog_manager.cc
M src/kudu/master/sentry_authz_provider-test.cc
M src/kudu/master/sentry_authz_provider.cc
M src/kudu/master/sentry_authz_provider.h
A src/kudu/master/sentry_client_metrics.cc
A src/kudu/master/sentry_client_metrics.h
M src/kudu/thrift/client.h
A src/kudu/thrift/ha_client_metrics.h
9 files changed, 284 insertions(+), 15 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/51/12951/2
-- 
To view, visit http://gerrit.cloudera.org:8080/12951
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I86e12ee6ee34f39c11f18b21e846cc5a1dee6b6f
Gerrit-Change-Number: 12951
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Tidy Bot (241)