Henry Robinson has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/2687 Tested-by: Internal Jenkins Reviewed-by: Henry Robinson <[email protected]> --- M be/src/transport/TSasl.cpp 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Henry Robinson: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2687 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d Gerrit-PatchSet: 4 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Frantisek Dvorak <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-Reviewer: Internal Jenkins
