Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/14881


Change subject: [master] a small clean-up on SysCatalogTable::SetupTablet()
......................................................................

[master] a small clean-up on SysCatalogTable::SetupTablet()

If master fails to replay orphaned operations from WAL, it crashes
due to the system tablet state check:

  F1206 01:32:15.488359 1324967 tablet_replica.cc:138] Check failed: state_ == 
SHUTDOWN || state_ == FAILED TabletReplica not fully shut down. State: 
BOOTSTRAPPING

This patch addresses the issue, so master would not crash at the
the tablet state consistency CHECK() under such conditions.  Instead,
now it reports corresponding error and crashes at the higher level
in master_main.cc.

I didn't add any tests: the replay of the orphaned transactions in WAL
during bootstrap has pretty good coverage already.

Change-Id: I6adfd7f74fdd2e05e04f6418cbf9bb86cad6465a
---
M src/kudu/kserver/kserver.cc
M src/kudu/master/master_main.cc
M src/kudu/master/sys_catalog.cc
3 files changed, 46 insertions(+), 40 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/81/14881/1
--
To view, visit http://gerrit.cloudera.org:8080/14881
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6adfd7f74fdd2e05e04f6418cbf9bb86cad6465a
Gerrit-Change-Number: 14881
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to