You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by as...@apache.org on 2014/11/10 18:36:08 UTC

incubator-sentry git commit: SENTRY-432: Fixing some pom and dependency issues

Repository: incubator-sentry
Updated Branches:
  refs/heads/sentry-hdfs-plugin e975aa4c4 -> d94bdb23e


SENTRY-432: Fixing some pom and dependency issues


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/d94bdb23
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/d94bdb23
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/d94bdb23

Branch: refs/heads/sentry-hdfs-plugin
Commit: d94bdb23eeeb9f484acf6f8b5085ffe55e8f6f05
Parents: e975aa4
Author: Arun Suresh <as...@cloudera.com>
Authored: Mon Nov 10 09:35:31 2014 -0800
Committer: Arun Suresh <as...@cloudera.com>
Committed: Mon Nov 10 09:35:31 2014 -0800

----------------------------------------------------------------------
 .../src/main/resources/sentry_hdfs_service.thrift         |  2 +-
 sentry-hdfs/sentry-hdfs-namenode-plugin/pom.xml           | 10 ----------
 .../apache/sentry/hdfs/UpdateableAuthzPermissions.java    |  7 ++++---
 3 files changed, 5 insertions(+), 14 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/d94bdb23/sentry-hdfs/sentry-hdfs-common/src/main/resources/sentry_hdfs_service.thrift
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/main/resources/sentry_hdfs_service.thrift b/sentry-hdfs/sentry-hdfs-common/src/main/resources/sentry_hdfs_service.thrift
index 9212b64..fb60855 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/main/resources/sentry_hdfs_service.thrift
+++ b/sentry-hdfs/sentry-hdfs-common/src/main/resources/sentry_hdfs_service.thrift
@@ -81,7 +81,7 @@ service SentryHDFSService
 {
   # HMS Path cache
   void handle_hms_notification(1:TPathsUpdate pathsUpdate);
-
+  i64 check_hms_seq_num(1:i64 pathSeqNum);
   TAuthzUpdateResponse get_all_authz_updates_from(1:i64 permSeqNum, 2:i64 pathSeqNum);
   map<string, list<string>> get_all_related_paths(1:string path, 2:bool exactMatch);
 }

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/d94bdb23/sentry-hdfs/sentry-hdfs-namenode-plugin/pom.xml
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-namenode-plugin/pom.xml b/sentry-hdfs/sentry-hdfs-namenode-plugin/pom.xml
index 4e5f6b0..813c2e4 100644
--- a/sentry-hdfs/sentry-hdfs-namenode-plugin/pom.xml
+++ b/sentry-hdfs/sentry-hdfs-namenode-plugin/pom.xml
@@ -35,16 +35,6 @@ limitations under the License.
       <version>1.5.0-incubating-SNAPSHOT</version>
     </dependency>
     <dependency>
-      <groupId>org.apache.sentry</groupId>
-      <artifactId>sentry-service-client</artifactId>
-      <version>1.5.0-incubating-SNAPSHOT</version>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.sentry</groupId>
-      <artifactId>sentry-hdfs-service</artifactId>
-      <version>1.5.0-incubating-SNAPSHOT</version>
-    </dependency>
-    <dependency>
       <groupId>junit</groupId>
       <artifactId>junit</artifactId>
       <scope>test</scope>

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/d94bdb23/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java b/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java
index e4ff1d5..e5af802 100644
--- a/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java
+++ b/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/UpdateableAuthzPermissions.java
@@ -27,7 +27,6 @@ import java.util.concurrent.locks.ReadWriteLock;
 
 import org.apache.hadoop.fs.permission.AclEntry;
 import org.apache.hadoop.fs.permission.FsAction;
-import org.apache.sentry.core.model.db.AccessConstants;
 import org.apache.sentry.hdfs.SentryPermissions.PrivilegeInfo;
 import org.apache.sentry.hdfs.SentryPermissions.RoleInfo;
 import org.apache.sentry.hdfs.service.thrift.TPrivilegeChanges;
@@ -46,8 +45,10 @@ public class UpdateableAuthzPermissions implements AuthzPermissions, Updateable<
   static {
     ACTION_MAPPING.put("ALL", FsAction.ALL);
     ACTION_MAPPING.put("*", FsAction.ALL);
-    ACTION_MAPPING.put(AccessConstants.SELECT.toUpperCase(), FsAction.READ_EXECUTE);
-    ACTION_MAPPING.put(AccessConstants.INSERT.toUpperCase(), FsAction.WRITE_EXECUTE);
+    ACTION_MAPPING.put("SELECT", FsAction.READ_EXECUTE);
+    ACTION_MAPPING.put("select", FsAction.READ_EXECUTE);
+    ACTION_MAPPING.put("INSERT", FsAction.WRITE_EXECUTE);
+    ACTION_MAPPING.put("insert", FsAction.WRITE_EXECUTE);
   }
 
   @Override