Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/16527
Change subject: WIP KUDU-2612 p??: initial implementation of TxnManager
......................................................................
WIP KUDU-2612 p??: initial implementation of TxnManager
This is a draft implementation of the TxnManager, still very raw
and WIP patch.
WIP:
* Collect some high-level initial feedback on the major points.
* Add many more tests, especially for BeginTransaction to make
txn_id allocation works as expected: as of now, it seems
we could get surprises until TxnStatusManager::BeginTransaction()
updated to avoid incrementing highest_txn_id_ and responding
back when the txn status tablet replica isn't a leader.
* Change the initialization of the TxnManager in master to
run on first txn-related request: that should fix the issue
of too many retries when masters are up and running, but
no tablet servers are around yet. Also, that should fix
many of currently failing tests scenarios which expect
there isn't any existing tables when they run right after
starting a mini cluster first time.
* Add more docs/comments.
* Cleanup the code.
Change-Id: Ie952977a3ae5f625d1283389f0be8afb79df7d8c
---
M src/kudu/integration-tests/ts_tablet_manager-itest.cc
M src/kudu/master/CMakeLists.txt
M src/kudu/master/catalog_manager.cc
M src/kudu/master/catalog_manager.h
M src/kudu/master/master.cc
M src/kudu/master/master.h
A src/kudu/master/txn_manager-test.cc
A src/kudu/master/txn_manager.cc
A src/kudu/master/txn_manager.h
A src/kudu/master/txn_manager.proto
A src/kudu/master/txn_manager_service.cc
A src/kudu/master/txn_manager_service.h
M src/kudu/transactions/txn_system_client.cc
13 files changed, 1,002 insertions(+), 15 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/27/16527/1
--
To view, visit http://gerrit.cloudera.org:8080/16527
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie952977a3ae5f625d1283389f0be8afb79df7d8c
Gerrit-Change-Number: 16527
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
