Todd Lipcon has submitted this change and it was merged. Change subject: Support downgrade to version that has LocalConsensus ......................................................................
Support downgrade to version that has LocalConsensus Continue writing the "local=false" field into RaftConfigPB when we have more than one member of the config. This partially reverts a few lines of code from revision 35a92b49913ceea80d9c0c61cb7af2ebcf007751, which removed the LocalConsensus implementation from Kudu. Because the "local" field will no longer be used, the code writing it was removed. However, since that field is used by previous versions, and defaults to true if unset, if we want to allow users to downgrade Kudu to an older version then we need to keep writing it for a while. Change-Id: I6d324b4f3ce5906bfc9a4df49dd60ca0cb919820 Reviewed-on: http://gerrit.cloudera.org:8080/3985 Reviewed-by: Todd Lipcon <t...@apache.org> Tested-by: Todd Lipcon <t...@apache.org> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/sys_catalog.cc 2 files changed, 8 insertions(+), 0 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/3985 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d324b4f3ce5906bfc9a4df49dd60ca0cb919820 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
