[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Move the maintenance manager to util .. Move the maintenance manager to util This moves the maintenance manager to util, along with its required p

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/3/src/kudu/util/maintenance_manager.h File src/k

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/3/src/kudu/util/maintenance_manager.h File src/kudu/util/

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3656 To unsubscribe, visit http://gerr

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2489/ -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3656 to look at the new patch set (#3). Change subject: Move the maintenance manager to util .. Move the maintenance man

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/2/src/kudu/tablet/CMakeLists.txt File src/kudu/t

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/2/src/kudu/tablet/CMakeLists.txt File src/kudu/ta

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: Move the maintenance manager to util .. Move the maintenance manager to util This moves the maintenance manager to util, along with its required protobufs. T

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2482/ -- To view, visit http://gerrit.cloudera.org:8080

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: (16 comments) This change is required as we'll need a server-wide MM for the result tracker. Added c

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/3656/1//COMMIT_MSG Commit Message: PS1, Line 9: it's

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: (15 comments) What's the motivation for this move? http://gerrit.cloudera.org:8080/#/c/3656/1//COMMIT_MSG Com

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/3656 Change subject: Move the maintenance manager to util .. Move the maintenance manager to util This moves the maintenance manager to

[kudu-CR] Move the maintenance manager to util

2016-07-14 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Move the maintenance manager to util .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2480/ -- To view, visit http://gerrit.cloudera.org:8080