Adar Dembo has submitted this change and it was merged. Change subject: master: include TS address in log messages ......................................................................
master: include TS address in log messages When looking at master logs, it's quite annoying to have to translate back from UUIDs to actual hostnames, since the operator typically wants to ssh into that node to look at logs, etc. This patch adds TSDescriptor::ToString() and calls it from all the points in CatalogManager where log messages refer to an individual server. This also adds validation that TS registrations must include at least one HTTP and one RPC address. This has always been the case but wasn't verified. Change-Id: Ic55fa7e818a115de70f9fc6aca12581c3b4779c7 Reviewed-on: http://gerrit.cloudera.org:8080/4131 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <a...@cloudera.com> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/ts_descriptor.cc M src/kudu/master/ts_descriptor.h M src/kudu/master/ts_manager.cc 4 files changed, 66 insertions(+), 50 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4131 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic55fa7e818a115de70f9fc6aca12581c3b4779c7 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>