You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2020/12/27 22:51:56 UTC
[Impala-ASF-CR] IMPALA-10027: configurable default anonymous user
Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/16902 )
Change subject: IMPALA-10027: configurable default anonymous user
......................................................................
IMPALA-10027: configurable default anonymous user
A username can be determined for a session via two mechanisms:
* In a secure env, the user is authenticated by LDAP or Kerberos
* In an unsecure env, the client specifies the user name, either
as a parameter to the OpenSession API (HS2) or as a parameter
to the first query run (beeswax)
This patch affects what happens if neither of the above mechanisms
is used. Previously we would end up with the username being an
empty string, but this makes Ranger unhappy. Hive uses the name
"anonymous" in this situation, so we change Impala's behaviour too.
This is configurable by -anonymous_user_name. -anonymous_user_name=
reverts to the old behaviour.
Test
* Add an end-to-end test that exercises this via impala-shell for
HS2, HS2-HTTP and beeswax protocols.
* Tweak a couple of existing tests that depended on the previous
behavior.
Change-Id: I6db491231fa22484aed476062b8fe4c8f69130b0
---
M be/src/common/global-flags.cc
M be/src/service/impala-beeswax-server.cc
M be/src/service/impala-hs2-server.cc
M fe/src/test/java/org/apache/impala/service/JdbcTest.java
M tests/custom_cluster/test_admission_controller.py
M tests/shell/test_shell_commandline.py
6 files changed, 44 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/02/16902/2
--
To view, visit http://gerrit.cloudera.org:8080/16902
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6db491231fa22484aed476062b8fe4c8f69130b0
Gerrit-Change-Number: 16902
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>