Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9180 )
Change subject: KUDU-2279 (part 5): don't include entity attributes in log ...................................................................... KUDU-2279 (part 5): don't include entity attributes in log Entity attributes like table names and partition descriptions aren't very useful in the context of the metrics log. They can always be grabbed from other places like 'kudu remote_replica list', etc, and are rarely relevant for tserver-scope debugging. Additionally, these might be considered somewhat sensitive for users to share publically in bug reports, etc, so may as well "redact" them. This shaves a few more bytes off the entries in the metrics log. Change-Id: Ic995f6a3b8430b73a041de8e5bcbc53a5d8e7f1b Reviewed-on: http://gerrit.cloudera.org:8080/9180 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <t...@apache.org> --- M src/kudu/server/server_base.cc M src/kudu/util/metrics.cc M src/kudu/util/metrics.h 3 files changed, 15 insertions(+), 6 deletions(-) Approvals: Kudu Jenkins: Verified Todd Lipcon: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9180 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic995f6a3b8430b73a041de8e5bcbc53a5d8e7f1b Gerrit-Change-Number: 9180 Gerrit-PatchSet: 4 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
