Jean-Daniel Cryans has posted comments on this change. ( http://gerrit.cloudera.org:8080/9172 )
Change subject: maintenance_manager: log the reason for scheduling each operation ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/9172/1/src/kudu/util/maintenance_manager.cc File src/kudu/util/maintenance_manager.cc: http://gerrit.cloudera.org:8080/#/c/9172/1/src/kudu/util/maintenance_manager.cc@444 PS1, Line 444: best performance improvement I'm worried what random users might read into this kind of log line. Also, would it make sense to print something related to the priority order instead of having to lookup the code? Maybe we'd still have to look at the code to see what the priority means. Just a thought. -- To view, visit http://gerrit.cloudera.org:8080/9172 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I4dcdb863a7a0b0fc2a72757801d5c057fa725c34 Gerrit-Change-Number: 9172 Gerrit-PatchSet: 1 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Wed, 31 Jan 2018 23:23:05 +0000 Gerrit-HasComments: Yes
