Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/11613 )
Change subject: [webui] KUDU-844 and other /tablet-rowsetlayout-svg improvements ...................................................................... Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/11613/3//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/11613/3//COMMIT_MSG@10 PS3, Line 10: show nit: shown http://gerrit.cloudera.org:8080/#/c/11613/3//COMMIT_MSG@37 PS3, Line 37: https://github.com/wdberkeley/kudu/blob/svg_screenshots/layout.png : : And here's how it looks without them: : : https://github.com/wdberkeley/kudu/blob/svg_screenshots/layout_no_text.png Maybe it's there and just not pictured, but is there also a count of the rowsets? Or maybe it's just not there and it'd be somewhat redundant given the svg? http://gerrit.cloudera.org:8080/#/c/11613/3/src/kudu/tablet/tablet.cc File src/kudu/tablet/tablet.cc: http://gerrit.cloudera.org:8080/#/c/11613/3/src/kudu/tablet/tablet.cc@2316 PS3, Line 2316: <div><p>In addition to the rowsets pictured and listed, " : "there are $0 rowset(s) currently undergoing compactions." : "</p></div> Should this be `num_rowsets_unavailable_for_compaction - 1` for the MRS? -- To view, visit http://gerrit.cloudera.org:8080/11613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7475f14093a187fde1329546b10c314da49fe08a Gerrit-Change-Number: 11613 Gerrit-PatchSet: 3 Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Mon, 08 Oct 2018 22:51:51 +0000 Gerrit-HasComments: Yes
