Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/15273 )
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
Reviewed-on: http://gerrit.cloudera.org:8080/15273
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
---
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(-)
Approvals:
Kudu Jenkins: Verified
Adar Dembo: Looks good to me, approved
--
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: merged
Gerrit-Change-Id: Ic172635a7c7b019e5f4394c6107345820d408a68
Gerrit-Change-Number: 15273
Gerrit-PatchSet: 4
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]>
