Hello Adar Dembo, Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/3985

to review the following change.

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
---
M src/kudu/master/catalog_manager.cc
M src/kudu/master/sys_catalog.cc
2 files changed, 8 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/85/3985/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3985
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6d324b4f3ce5906bfc9a4df49dd60ca0cb919820
Gerrit-PatchSet: 1
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>

Reply via email to