Will Berkeley has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11290 )

Change subject: [webui] Make the tables on /tablets fancy
......................................................................

[webui] Make the tables on /tablets fancy

The /tablets page has live tablets and a tombstoned tablets tables.
These tables get big and unwieldy to use. This patch makes the tables
paginated and searchable by adding the bootstrap-table plugin[1]. I
picked this plugin because it integrates with our existing bootstrap
styles, it worked out of the box with no effort, and the project looks
active. I would've liked a plugin that could be used to sort columns
based on their type out of the box, but I couldn't find a good
candidate.

I also removed a couple of columns that I've never seen be useful:

* Write buffer memory usage: Users are not usually interested in this for
  specific tablets, except possibly to identify outliers, which can be
  done on the mem-trackers page.
* Last status: This is updated at tablet creation or bootstrap time and
  never contains useful information.

Less columns allows the table to fit comfortably on smaller screens.

1: https://github.com/wenzhixin/bootstrap-table

Change-Id: I5243e9cb839e75936c71b14fed08881317ee0f5f
Reviewed-on: http://gerrit.cloudera.org:8080/11290
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Will Berkeley <[email protected]>
---
M build-support/release/rat_exclude_files.txt
M src/kudu/server/webserver.cc
M src/kudu/tserver/tserver_path_handlers.cc
A www/bootstrap/css/bootstrap-table.min.css
A www/bootstrap/js/bootstrap-table.min.js
M www/tablets.mustache
6 files changed, 67 insertions(+), 62 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Will Berkeley: Verified

--
To view, visit http://gerrit.cloudera.org:8080/11290
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5243e9cb839e75936c71b14fed08881317ee0f5f
Gerrit-Change-Number: 11290
Gerrit-PatchSet: 5
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>

Reply via email to