You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Thomas Tauber-Marshall (Code Review)" <ge...@cloudera.org> on 2016/05/18 18:35:24 UTC

[Impala-CR](cdh5-trunk) IMPALA-3542: do_as_user empty check missing

Thomas Tauber-Marshall has uploaded a new change for review.

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

Change subject: IMPALA-3542: do_as_user empty check missing
......................................................................

IMPALA-3542: do_as_user empty check missing

Fixes a typo in ImpalaServer::AuthorizeProxyUser where we
check that the 'user' parameters isn't empty twice instead
of also checking the 'do_as_user' parameter.

Change-Id: I8e3962f6f397804e37d4f2c667e97b55bd3ca2bf
---
M be/src/service/impala-server.cc
M tests/hs2/test_hs2.py
2 files changed, 17 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/19/3119/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3119
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8e3962f6f397804e37d4f2c667e97b55bd3ca2bf
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Thomas Tauber-Marshall <tm...@cloudera.com>