Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9266 )
Change subject: WIP: KUDU-2274. RaftConsensus should not access cmeta when shutdown ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/9266/1/src/kudu/integration-tests/ts_tablet_manager-itest.cc File src/kudu/integration-tests/ts_tablet_manager-itest.cc: http://gerrit.cloudera.org:8080/#/c/9266/1/src/kudu/integration-tests/ts_tablet_manager-itest.cc@411 PS1, Line 411: ASSERT_OK(consensus->ConsensusState(&cs, RaftConsensus::INCLUDE_HEALTH_REPORT)); It seems this assertion fails because it's not possible to fetch consensus state from a failed replica: /home/jenkins-slave/workspace/kudu-master/2/src/kudu/integration-tests/ts_tablet_manager-itest.cc:411: Failure Failed Bad status: Service unavailable: Tablet replica is shutdown /home/jenkins-slave/workspace/kudu-master/2/src/kudu/integration-tests/ts_tablet_manager-itest.cc:520: Failure Expected: get_health_reports(&reports) doesn't generate new fatal failures in the current thread. Actual: it does. /home/jenkins-slave/workspace/kudu-master/2/src/kudu/util/test_util.cc:291: Failure Failed -- To view, visit http://gerrit.cloudera.org:8080/9266 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib3f3f75674d5739b281e7c689ddb2d433b9b7415 Gerrit-Change-Number: 9266 Gerrit-PatchSet: 1 Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Fri, 09 Feb 2018 18:28:12 +0000 Gerrit-HasComments: Yes