Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14217 )
Change subject: KUDU-2069 p1: add persistent tserver maintenance mode ...................................................................... KUDU-2069 p1: add persistent tserver maintenance mode Adds tablet server states to the master, with which to represent tablet server maintenance mode (and in the future, decommissioning). The on-disk state is represented as an entry in the system catalog table, keyed by the tserver UUID. The in-memory state is maintained by the TSManager as a mapping from tserver UUID to the new TServerStatePB enum. Note that this patch only introduces the states internally and doesn't expose a way for users to set them. Testing: - test to check that maintenance mode is maintained through a restart of the master - test to ensure that repeated setting of state does not affect the in-memory tserver states - test that concurrently updates tserver state and ensures that the results in-memory match with what's on-disk Change-Id: Ib669b43b3cee171c4c7dbd54041e29c30cb9f767 Reviewed-on: http://gerrit.cloudera.org:8080/14217 Reviewed-by: Adar Dembo <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/master/CMakeLists.txt M src/kudu/master/catalog_manager.cc M src/kudu/master/master.proto M src/kudu/master/sys_catalog.cc M src/kudu/master/sys_catalog.h M src/kudu/master/ts_manager.cc M src/kudu/master/ts_manager.h A src/kudu/master/ts_state-test.cc 8 files changed, 444 insertions(+), 21 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14217 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib669b43b3cee171c4c7dbd54041e29c30cb9f767 Gerrit-Change-Number: 14217 Gerrit-PatchSet: 9 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
