Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13774 )

Change subject: Support SPNEGO for Impala webserver
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/13774/1/be/src/util/webserver.cc
File be/src/util/webserver.cc:

http://gerrit.cloudera.org:8080/#/c/13774/1/be/src/util/webserver.cc@203
PS1, Line 203: // is invalid, a bad Status will be returned (and the other 
out-parameters left untouched).
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/13774/1/be/src/util/webserver.cc@214
PS1, Line 214:   RETURN_NOT_OK(kudu::gssapi::SpnegoStep(neg_token, 
&resp_token_b64, &is_complete, authn_user));
line too long (96 > 90)



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ife2b04310e1571d231bf8ee1bcfd3b7afc2edd8f
Gerrit-Change-Number: 13774
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Thomas Tauber-Marshall <tmarsh...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Comment-Date: Tue, 02 Jul 2019 00:03:16 +0000
Gerrit-HasComments: Yes

Reply via email to