Hello Dan Burkert, Adar Dembo, I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/4131 to review the following change. 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 --- 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, 45 insertions(+), 35 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/31/4131/1 -- To view, visit http://gerrit.cloudera.org:8080/4131 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic55fa7e818a115de70f9fc6aca12581c3b4779c7 Gerrit-PatchSet: 1 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>