Hello Mike Percy, Alexey Serbin, Dan Burkert, Kudu Jenkins, Todd Lipcon,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8089
to look at the new patch set (#8).
Change subject: catalog_manager: make ScopedTabletInfoCommitter generic
......................................................................
catalog_manager: make ScopedTabletInfoCommitter generic
I want to use it for TableInfo mutations, and to manage multiple read locks
too. So I've moved it to util/cow_object, created a read-only variant, and
templatized them for any CowObject.
Importantly, the default out-of-scope behavior is now Unlock(). Previously it
was Commit(), but I think Unlock() is less surprising.
Change-Id: I890db0fd18f773ab2253c49817def8162e04ad25
---
M src/kudu/integration-tests/create-table-stress-test.cc
M src/kudu/master/catalog_manager-test.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
M src/kudu/master/master-path-handlers.cc
M src/kudu/master/sys_catalog-test.cc
M src/kudu/master/sys_catalog.cc
M src/kudu/util/CMakeLists.txt
A src/kudu/util/cow_object.cc
M src/kudu/util/cow_object.h
10 files changed, 436 insertions(+), 306 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/89/8089/8
--
To view, visit http://gerrit.cloudera.org:8080/8089
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I890db0fd18f773ab2253c49817def8162e04ad25
Gerrit-Change-Number: 8089
Gerrit-PatchSet: 8
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
