Adar Dembo has posted comments on this change.

Change subject: server: move apply_pool into ServerBase
......................................................................


Patch Set 4:

There were two interesting TSAN warnings here.
1. ksck_remote-test: this TSAN warning is about a data race in 
GetKuduKerberosPrincipalOidNid(), which is Kudu's certificate handling code. 
Unrelated to this patch.
2. tablet_replica-test: this one looks like it's related to the "consolidate 
Raft thread pool" patch. Need to think about it some more.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie7ffc886269aa6531517a52fef29c4408a925aed
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to