Yifan Zhang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19407 )
Change subject: [web] add maintenance op statistics information on web pages for data_retained_bytes ...................................................................... [web] add maintenance op statistics information on web pages for data_retained_bytes MaintenanceOpStats collects statistics about 'data_retained_bytes_', which means amount of disk capacity in data blocks that would be freed if current maintenence operation ran. But the protobuf file has no such information, so it is not shown on web pages. This patch adds a field in the protobuf file and updates the embedded web server to show the retained bytes for a maintenance operation. Sample screenshot of the maintenance manager page can be found here: https://github.com/shenxingwuying/kudu/blob/dev/flyline/screenshot/data_retained.png Change-Id: Iac8f0307705d57cea48901102b170c88d73d8c2e Reviewed-on: http://gerrit.cloudera.org:8080/19407 Tested-by: Kudu Jenkins Reviewed-by: Yifan Zhang <[email protected]> --- M src/kudu/integration-tests/webserver-stress-itest.cc M src/kudu/tserver/tserver_path_handlers.cc M src/kudu/util/maintenance_manager.cc M src/kudu/util/maintenance_manager.proto M www/maintenance-manager.mustache 5 files changed, 28 insertions(+), 1 deletion(-) Approvals: Kudu Jenkins: Verified Yifan Zhang: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/19407 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iac8f0307705d57cea48901102b170c88d73d8c2e Gerrit-Change-Number: 19407 Gerrit-PatchSet: 8 Gerrit-Owner: Yuqi Du <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Ashwani Raina <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Yuqi Du <[email protected]>
