Will Berkeley has posted comments on this change. Change subject: [webui] Improvements for when there's many tablets & cleanup ......................................................................
Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6870/1//COMMIT_MSG Commit Message: Line 17: 4. Detailed tablet tables use table-hover so it's easier to > 5. Done http://gerrit.cloudera.org:8080/#/c/6870/1/src/kudu/master/master-path-handlers.cc File src/kudu/master/master-path-handlers.cc: Line 256: TabletMetadataLock l(tablet.get(), TabletMetadataLock::READ); > Would it be possible to combine all of this with the work in L278+ to avoid Done http://gerrit.cloudera.org:8080/#/c/6870/1/src/kudu/server/webserver.cc File src/kudu/server/webserver.cc: Line 424: " <script src='/jquery-1.11.1.min.js' defer></script>" > I noticed we include this already in www/metrics.html. Should we remove it They get different headers-- metrics.html's is hard-coded. http://gerrit.cloudera.org:8080/#/c/6870/1/src/kudu/tserver/tserver-path-handlers.cc File src/kudu/tserver/tserver-path-handlers.cc: PS1, Line 243: replica->tablet()->mem_tracker() > Can this actually be null? Done -- To view, visit http://gerrit.cloudera.org:8080/6870 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic3904a4b0fbb6446615cd46c8a6f30f81c832c53 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com> Gerrit-HasComments: Yes