Hello Alexey Serbin, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11290
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/90/11290/3
--
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: newpatchset
Gerrit-Change-Id: I5243e9cb839e75936c71b14fed08881317ee0f5f
Gerrit-Change-Number: 11290
Gerrit-PatchSet: 3
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <[email protected]>