Hello Tidy Bot, Attila Bukor, Yifan Zhang, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/19389
to look at the new patch set (#2).
Change subject: [server] add 'memory_usage' metric for server
......................................................................
[server] add 'memory_usage' metric for server
Recently, from a post-mortem analysis of diagnostic logs created by
kudu-tserver process, I realized it would be useful to know current
memory consumption along with other metrics and stack traces of all
the threads run by the process.
I didn't add any tests for the newly introduced metric, but I manually
verified that both kudu-master and kudu-tserver processes properly
reported current memory usage among other server metrics at the
"/metrics" HTTP endpoint. The newly introduced metric was also present
in the diagnostic log files, and it was possible to extract the readings
of the metric by running the following kudu CLI command on the
corresponding files:
kudu diagnose parse_metrics --simple_metrics=server.memory_usage
Change-Id: If066be928bd1c7c3f37c875937ae8802dd8fcd33
---
M src/kudu/server/server_base.cc
1 file changed, 16 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/89/19389/2
--
To view, visit http://gerrit.cloudera.org:8080/19389
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If066be928bd1c7c3f37c875937ae8802dd8fcd33
Gerrit-Change-Number: 19389
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yifan Zhang <[email protected]>
