You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by el...@apache.org on 2016/09/11 19:28:36 UTC
[7/7] phoenix git commit: PHOENIX-3176 Remove unnecessary PQS proxy
user authorization
PHOENIX-3176 Remove unnecessary PQS proxy user authorization
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo
Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/7412f2a1
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/7412f2a1
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/7412f2a1
Branch: refs/heads/4.8-HBase-0.98
Commit: 7412f2a11f223b573cd927c53af02b5d641fdad4
Parents: 71b0b62
Author: Josh Elser <el...@apache.org>
Authored: Sat Sep 10 16:05:15 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Sun Sep 11 15:25:56 2016 -0400
----------------------------------------------------------------------
.../main/java/org/apache/phoenix/queryserver/server/Main.java | 4 ----
1 file changed, 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/phoenix/blob/7412f2a1/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
----------------------------------------------------------------------
diff --git a/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java b/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
index 4b3ca7e..0ed3b7b 100644
--- a/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
+++ b/phoenix-queryserver/src/main/java/org/apache/phoenix/queryserver/server/Main.java
@@ -286,10 +286,6 @@ public final class Main extends Configured implements Tool, Runnable {
// LoadingCache will create a new instance for us if one isn't cached.
UserGroupInformation proxyUser = createProxyUser(remoteUserName);
- // Check if this user is allowed to be impersonated.
- // Will throw AuthorizationException if the impersonation as this user is not allowed
- ProxyUsers.authorize(proxyUser, remoteAddress);
-
// Execute the actual call as this proxy user
return proxyUser.doAs(new PrivilegedExceptionAction<T>() {
@Override