Alexey Serbin has posted comments on this change.

Change subject: [security-faults-itest] added another krb5 error message
......................................................................


Patch Set 2:

> oh, I misunderstood your earlier comment. I thought _we_ were the
 > ones calling that function.
 > 
 > One more reason we should drop cyrus-sasl at some point and just
 > use GSSAPI directly, I guess.

Sorry for the confusion.

Yep, I think this is just another minor thing after that major 
performance-related bottleneck.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I88462a186bd77ff6102a2a07ab3e885d704c3bb3
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to