Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/9574 )
Change subject: KUDU-2295 fix nullptr dereference in Tablet ...................................................................... Patch Set 2: > Would be nice to have a test for this in the codebase, since it > sounds like you had to make some local modifications to > raft_consensus-stress-itest to cause it. Perhaps in > tablet_server-stress-test or delete_tablet-itest we could add > something more specific to this issue? If I enable those modifications, the test fails because of the assert in TestWorkload. There is specific TODO in raft_consensus_stress-itest. Once the issue with that addressed (via retries in TestWorkload or via leader replicas), I'm going to enable that additional reader thread. -- To view, visit http://gerrit.cloudera.org:8080/9574 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib4b8cc2a307e5a0bd6019bd155f33c1565fca513 Gerrit-Change-Number: 9574 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Mon, 12 Mar 2018 23:37:50 +0000 Gerrit-HasComments: No
