incubator-kudu git commit: Add port in web ui tables, add role and table name to /tablet page

[jd...@apache.org]

Repository: incubator-kudu
Updated Branches:
  refs/heads/master d0cff255f -> 17e2ae570


Add port in web ui tables, add role and table name to /tablet page

Couple of improvements to web ui:
1. The RaftConfig column in the tablet table on master and tserver now
displays the port as well as the hostname for the tablet.
2. The /tablet page displays the tablet role and the table name.

Change-Id: Ia80b74346cae1a75d66b400521e07aa1994d1d65
Reviewed-on: http://gerrit.cloudera.org:8080/3553
Tested-by: Kudu Jenkins
Reviewed-by: Jean-Daniel Cryans <jd...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/17e2ae57
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/17e2ae57
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/17e2ae57

Branch: refs/heads/master
Commit: 17e2ae570b825a54d793e6537acca8f9cb7f8f54
Parents: d0cff25
Author: Will Berkeley <wd...@gmail.com>
Authored: Fri Jul 1 12:25:02 2016 -0400
Committer: Jean-Daniel Cryans <jd...@apache.org>
Committed: Fri Jul 15 23:35:43 2016 +0000

----------------------------------------------------------------------
 src/kudu/master/master-path-handlers.cc   |  5 +++--
 src/kudu/tserver/tserver-path-handlers.cc | 10 ++++++++--
 2 files changed, 11 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/17e2ae57/src/kudu/master/master-path-handlers.cc
----------------------------------------------------------------------
diff --git a/src/kudu/master/master-path-handlers.cc b/src/kudu/master/master-path-handlers.cc
index e286bb2..d3f1c0f 100644
--- a/src/kudu/master/master-path-handlers.cc
+++ b/src/kudu/master/master-path-handlers.cc
@@ -448,11 +448,12 @@ string MasterPathHandlers::TSDescriptorToHtml(const TSDescriptor& desc,
   desc.GetRegistration(&reg);
 
   if (reg.http_addresses().size() > 0) {
-    return Substitute("<a href=\"http://$0:$1/tablet?id=$2\">$3</a>",
+    return Substitute("<a href=\"http://$0:$1/tablet?id=$2\">$3:$4</a>",
                       reg.http_addresses(0).host(),
                       reg.http_addresses(0).port(),
                       EscapeForHtmlToString(tablet_id),
-                      EscapeForHtmlToString(reg.http_addresses(0).host()));
+                      EscapeForHtmlToString(reg.http_addresses(0).host()),
+                      reg.http_addresses(0).port());
   } else {
     return EscapeForHtmlToString(desc.permanent_uuid());
   }

http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/17e2ae57/src/kudu/tserver/tserver-path-handlers.cc
----------------------------------------------------------------------
diff --git a/src/kudu/tserver/tserver-path-handlers.cc b/src/kudu/tserver/tserver-path-handlers.cc
index 6cad63f..19c19a3 100644
--- a/src/kudu/tserver/tserver-path-handlers.cc
+++ b/src/kudu/tserver/tserver-path-handlers.cc
@@ -248,7 +248,10 @@ string TabletServerPathHandlers::ConsensusStatePBToHtml(const ConsensusStatePB&
   std::sort(sorted_peers.begin(), sorted_peers.end(), &CompareByMemberType);
   for (const RaftPeerPB& peer : sorted_peers) {
     string peer_addr_or_uuid =
-        peer.has_last_known_addr() ? peer.last_known_addr().host() : peer.permanent_uuid();
+        peer.has_last_known_addr() ? Substitute("$0:$1",
+                                                peer.last_known_addr().host(),
+                                                peer.last_known_addr().port())
+                                   : peer.permanent_uuid();
     peer_addr_or_uuid = EscapeForHtmlToString(peer_addr_or_uuid);
     string role_name = RaftPeerPB::Role_Name(GetConsensusRole(peer.permanent_uuid(), cstate));
     string formatted = Substitute("$0: $1", role_name, peer_addr_or_uuid);
@@ -314,8 +317,11 @@ void TabletServerPathHandlers::HandleTabletPage(const Webserver::WebRequest& req
   if (!LoadTablet(tserver_, req, &tablet_id, &peer, output)) return;
 
   string table_name = peer->tablet_metadata()->table_name();
+  RaftPeerPB::Role role = peer->consensus()->role();
 
-  *output << "<h1>Tablet " << EscapeForHtmlToString(tablet_id) << "</h1>\n";
+  *output << "<h1>Tablet " << EscapeForHtmlToString(tablet_id)
+          << " (" << RaftPeerPB::Role_Name(role) << ")</h1>\n";
+  *output << "<h3>Table " << EscapeForHtmlToString(table_name) << "</h3>";
 
   // Output schema in tabular format.
   *output << "<h2>Schema</h2>\n";