You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@impala.apache.org by "Frantisek Dvorak (Code Review)" <ge...@cloudera.org> on 2016/03/31 23:56:11 UTC

[Impala-CR](cdh5-trunk) IMPALA-2645: Set mechUsing only on success

Frantisek Dvorak has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2687

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/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2687
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I71e5bb754866003e2a282ce3eeb8d4e6eac0573d
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Frantisek Dvorak <va...@civ.zcu.cz>