Hmm I didn't test upgrade for (2), I wonder if it would actually fail upgrade. I'm on a plane right now but I can try running an upgrade once I land.
Sent from my iPhone On Aug 19, 2016, at 9:23 AM, Dinesh Bhat <[email protected]> wrote: >> >> 1) The is_local() flag default is actually ignored by 0.9.1 and it has a >> CHECK to ensure that the field is set in the RaftConfigPB. That means >> that 74210b2546df9fd5dec7bb926eeb524362d2da90 was not a sufficient fix for >> backcompat. Fix: https://gerrit.cloudera.org/4059 to fix it "again". >> 2) Adding a field to TSRegistrationPB in KUDU-1490 triggered some error >> validation in TSDescriptor::Register() that the PB will not change between >> registration invocations. I tested reverting KUDU-1490 and this appeared to >> solve the problem. The revert is here: https://gerrit.cloudera.org/4060 >> >> If we want to maintain downgradability for this release then we could apply >> these patches and do a quick re-vote without the waiting period, perhaps? > > Thanks for catching these Mike. Regarding 2), I vote to revert the change in > 0.10.0 reason being, we anyways have to take care of this version for Masters > too ( KUDU-1534 <https://issues.apache.org/jira/browse/KUDU-1534> ). We can > address validation/compatibility criteria for both of them together and > integrate them post 0.10.0. > > Tx, > ./dinesh.
