Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/16973 )
Change subject: [tablet] OpDriver::ExecuteAsync() return void ...................................................................... Patch Set 1: > LGTM. Not sure what's up with the unit tests, but it doesn't seem > like it could be related. Thank you for review. Yep, I looked at the failures and re-run those tests via dist-test, and didn't see failures. From the semantics of this patch and my recent verification it's clear those failures are unrelated. -- To view, visit http://gerrit.cloudera.org:8080/16973 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I459cdac9f17a6345b2508a6b7137c0768811dd87 Gerrit-Change-Number: 16973 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Comment-Date: Mon, 25 Jan 2021 22:09:52 +0000 Gerrit-HasComments: No
