Adar Dembo has posted comments on this change. Change subject: Move maintenance_manager to server/ ......................................................................
Patch Set 1: > hrm, maybe we should just bite the bullet and make src/maintenance? > My issue with 'common' is that that currently refers to > "data-model" type stuff common between client and server, whereas > the MM is server-only. I understand the concern with common/. I'd be OK with maintenance/, though FWIW I think that suggests a finer-grained modularization scheme than we currently adhere to (since there's not much MM code); I'm OK with that too. -- To view, visit http://gerrit.cloudera.org:8080/6506 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8f89fa0e1ffa7304e29262edfab21b49d8b91665 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: No
