Adar Dembo has submitted this change and it was merged. Change subject: KUDU-495 (part 1): new sys_catalog writing API ......................................................................
KUDU-495 (part 1): new sys_catalog writing API The new API allows any arbitrary combination of table or tablet creation/deletion. With some additional changes, we won't need to "roll forward" any persistent state following a master failover. There are no functional changes to the master here; just API updates. I was hoping to use designated initializer lists to set up WriteActions, but apparently it's a C-only feature [1]. 1. http://stackoverflow.com/q/18731707 Change-Id: I7f5e5ecbff33435c964ced2e421566c41c387e9e Reviewed-on: http://gerrit.cloudera.org:8080/2648 Tested-by: Kudu Jenkins Reviewed-by: David Ribeiro Alves <[email protected]> --- M src/kudu/master/catalog_manager.cc M src/kudu/master/sys_catalog-test.cc M src/kudu/master/sys_catalog.cc M src/kudu/master/sys_catalog.h 4 files changed, 138 insertions(+), 120 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2648 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7f5e5ecbff33435c964ced2e421566c41c387e9e Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
