[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-03-05 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9172 ) Change subject: maintenance_manager: log the reason for scheduling each operation .. maintenance_manager: log the reason for

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-03-05 Thread Will Berkeley (Code Review)
Will Berkeley 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 2: Code-Review+2 -- To view, visit

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-03-04 Thread Todd Lipcon (Code Review)
Todd Lipcon 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 2: Any thoughts on this? Would like to

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-02-23 Thread Todd Lipcon (Code Review)
Todd Lipcon 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: OK, I made things a bit more concise,

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-02-23 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Jean-Daniel Cryans, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9172 to look at the new patch set (#2). Change subject: maintenance_manager: log the reason for scheduling each operation

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-02-02 Thread Will Berkeley (Code Review)
Will Berkeley 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)

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-01-31 Thread Jean-Daniel Cryans (Code Review)
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)

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-01-31 Thread Todd Lipcon (Code Review)
Todd Lipcon 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)

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-01-31 Thread Jean-Daniel Cryans (Code Review)
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)

[kudu-CR] maintenance manager: log the reason for scheduling each operation

2018-01-31 Thread Todd Lipcon (Code Review)
Hello Will Berkeley, Jean-Daniel Cryans, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9172 to review the following change. Change subject: maintenance_manager: log the reason for scheduling each operation