Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16034 )

Change subject: [consensus] KUDU-2727 lock-free CheckLeadershipAndBindTerm()
......................................................................


Patch Set 6:

> Seems you have benchmarked this here:
 >
 > https://gerrit.cloudera.org/c/16060/

Right: that test was written specifically to trigger conditions this patch 
addresses.  The write request rate to the tablet is kept the same (because all 
those requests are blocked on a slowed WAL), but the  lock contention is gone 
and RPC queue has capacity to accept other requests.

Thank you for review!


--
To view, visit http://gerrit.cloudera.org:8080/16034
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I934ae3035d893fd850afe27d96f8dd6612c9ffbd
Gerrit-Change-Number: 16034
Gerrit-PatchSet: 6
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Mon, 22 Jun 2020 18:55:25 +0000
Gerrit-HasComments: No

Reply via email to