[ranger] branch master updated: Revert "RANGER-2891: Add checkCanShowColumnsMetadata for presto plugin"

[pradeep]

This is an automated email from the ASF dual-hosted git repository.

pradeep pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new e799673  Revert "RANGER-2891: Add checkCanShowColumnsMetadata for 
presto plugin"
e799673 is described below

commit e79967389edf007ce181553e5ade8d5912c3a8f4
Author: pradeep <prad...@apache.org>
AuthorDate: Mon Jul 6 15:49:39 2020 +0530

    Revert "RANGER-2891: Add checkCanShowColumnsMetadata for presto plugin"
    
    This reverts commit 0fa35a1d00de7899effa5a589713baf947cabd2b.
---
 .../presto/authorizer/RangerSystemAccessControl.java           |  8 --------
 .../presto/authorizer/RangerSystemAccessControl.java           | 10 ----------
 2 files changed, 18 deletions(-)

diff --git 
a/plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
 
b/plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
index b6409cf..f4fc89d 100644
--- 
a/plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
+++ 
b/plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
@@ -639,14 +639,6 @@ public class RangerSystemAccessControl
     }
   }
 
-  @Override
-  public void checkCanShowColumnsMetadata(SystemSecurityContext context, 
CatalogSchemaTableName table) {
-    if (!hasPermission(createResource(table), context, 
PrestoAccessType.SELECT)) {
-      LOG.debug("RangerSystemAccessControl.checkCanShowColumnsMetadata(" + 
table.getSchemaTableName().getTableName() + ") denied");
-      
AccessDeniedException.denyShowColumnsMetadata(table.getSchemaTableName().getTableName());
-    }
-  }
-
   /** HELPER FUNCTIONS **/
 
   private RangerPrestoAccessRequest createAccessRequest(RangerPrestoResource 
resource, SystemSecurityContext context, PrestoAccessType accessType) {
diff --git 
a/ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
 
b/ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
index f132355..4063870 100644
--- 
a/ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
+++ 
b/ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
@@ -526,16 +526,6 @@ public class RangerSystemAccessControl
     }
   }
 
-  @Override
-  public void checkCanShowColumnsMetadata(SystemSecurityContext 
systemSecurityContext, CatalogSchemaTableName table){
-    try {
-      activatePluginClassLoader();
-      
systemAccessControlImpl.checkCanShowColumnsMetadata(systemSecurityContext, 
table);
-    } finally {
-      deactivatePluginClassLoader();
-    }
-  }
-
   private void activatePluginClassLoader() {
     if (rangerPluginClassLoader != null) {
       rangerPluginClassLoader.activate();