Todd Lipcon 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 -- 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: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: No
