Alexey Serbin has posted comments on this change.

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


Patch Set 2:

> > are you working on a separate patch to try to get rid of that
 > > '(Success)' thing?
 > 
 > I'm not: it seems that that 'Success' comes from the krb5 library
 > as the minor error status message extracted by the
 > gss_display_status() function.  I'm not sure whether we want to fix
 > that.

I just looked at sasl_gss_seterror_() function from cyrus-sasl library in 
plugins/gssapi.c.

-- 
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