> > 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.
