Hello Kudu Jenkins, Adar Dembo, Volodymyr Verovkin,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15273
to look at the new patch set (#3).
Change subject: [clock] safer handling of metrics-related endpoints
......................................................................
[clock] safer handling of metrics-related endpoints
Re-worked the way of retrieving hybrid clock metrics:
* in case of 'system' time source, do not crash if clock is not
synchronized when serving /metrics Web server endpoint
* in general, register function gauges only after HybridClock
is initialized
In addition, this patch contains a new test scenario to exercise the
control path mentioned above.
This patch also contains a sub-scenario to check for the presence of
metrics introduced in the context of KUDU-3048.
This is a follow-up for 8808b041c, 6371ffdb5, and 7d9d7009a.
Change-Id: Ic172635a7c7b019e5f4394c6107345820d408a68
---
M src/kudu/clock/CMakeLists.txt
M src/kudu/clock/hybrid_clock-test.cc
M src/kudu/clock/hybrid_clock.cc
M src/kudu/clock/hybrid_clock.h
M src/kudu/integration-tests/webserver-stress-itest.cc
M src/kudu/tserver/tablet_server-test.cc
6 files changed, 201 insertions(+), 80 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/73/15273/3
--
To view, visit http://gerrit.cloudera.org:8080/15273
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic172635a7c7b019e5f4394c6107345820d408a68
Gerrit-Change-Number: 15273
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Volodymyr Verovkin <[email protected]>
