Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/14259 )

Change subject: KUDU-2946: avoid ServicePool inc/dec in 
Messenger::QueueInboundCall
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/14259/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/14259/1//COMMIT_MSG@11
PS1, Line 11: a
> nit: a
Done


http://gerrit.cloudera.org:8080/#/c/14259/1/src/kudu/rpc/messenger.cc
File src/kudu/rpc/messenger.cc:

http://gerrit.cloudera.org:8080/#/c/14259/1/src/kudu/rpc/messenger.cc@375
PS1, Line 375:   // This lock acquisition spans the entirety of the function to 
avoid having to
> +1
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Id759617dcb13a2533e9ce071880c43678b700d25
Gerrit-Change-Number: 14259
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 19 Sep 2019 00:13:36 +0000
Gerrit-HasComments: Yes

Reply via email to