[kudu-CR] KUDU-495 (part 1): new sys_catalog writing API

["Adar Dembo (Code Review)" <ge...@cloudera.org>]

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 <da...@cloudera.com>
---
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 <ad...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>