Todd Lipcon has posted comments on this change.

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


Patch Set 1:

(1 comment)

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

Line 13:   'GSSAPI Error: The referenced context has expired (Success)'
> ... could get 'minor error status' from the krb5 library using  gss_display
yea, I think if we don't have any extra detail/message to add, we should just 
add nothing. Adding (Success) is very confusing to users in the context of an 
error message, and saying "we couldn't tell you anything more than this!" is 
not very helpful either.


-- 
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: 1
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: Yes

Reply via email to