Alexey Serbin has posted comments on this change. Change subject: [master] CA management: added SysTable base class ......................................................................
Patch Set 1: > Unfortunately I don't think this approach is going to work. > > We originally had something like this, with separate catalog > tablets for 'tablets' and 'tables' metadata. But then when we got > to doing HA, we realized it didn't really work because we needed > the concept of which master was "the leader", and if you have more > than one tablet, it's quite possible that leadership duties are > split between them, making a lot of logic more complicated. > > So, rather than do this, I think we should just extend the current > sys_catalog with another protobuf entry type. Our design is a bit > ugly in that we are using our lovely schemaful structured storage > mechanism to store protobuf blobs in a key/value schema, but it > simplifies the HA story quite a bit. > > If you want to see the historical context, check out commit > 77fab0e20c8641aea751daa5ce05eae879fcec09 Todd, thank you for the analysis and the reference. I'll amend this patch as you suggest. -- To view, visit http://gerrit.cloudera.org:8080/5063 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I00ccb45cd01cc63b044e5ffe4b3e194ae68cdb66 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: No