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 (#4).

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/4
--
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: 4
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]>

Reply via email to