Hello Internal Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/2687
to look at the new patch set (#3).
Change subject: IMPALA-2645: Set mechUsing only on success
......................................................................
IMPALA-2645: Set mechUsing only on success
Better to check success status from sasl_client_start() before using
returned structure with auth mechanism. According to the API
documentation, its value is guaranteed only after success (no error, or
SASL_CONTINUE).
Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d
---
M be/src/transport/TSasl.cpp
1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/87/2687/3
--
To view, visit http://gerrit.cloudera.org:8080/2687
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Frantisek Dvorak <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Internal Jenkins