sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lsk...@apache.org
Subject incubator-sentry git commit: SENTRY-545: Disable privilege cleanup by default (Mike Yoder via Lenni Kuff)
Date Mon, 01 Dec 2014 20:02:29 GMT
Repository: incubator-sentry
Updated Branches:
  refs/heads/master 1a697a090 -> 3d3f96ca7


SENTRY-545: Disable privilege cleanup by default (Mike Yoder via Lenni Kuff)


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

Branch: refs/heads/master
Commit: 3d3f96ca7939e687944f1426419db673f8797f3e
Parents: 1a697a0
Author: Lenni Kuff <lskuff@cloudera.com>
Authored: Mon Dec 1 12:02:06 2014 -0800
Committer: Lenni Kuff <lskuff@cloudera.com>
Committed: Mon Dec 1 12:02:06 2014 -0800

----------------------------------------------------------------------
 .../java/org/apache/sentry/service/thrift/ServiceConstants.java   | 2 +-
 .../sentry/provider/db/service/persistent/TestSentryStore.java    | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/3d3f96ca/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
index eb92c1e..06aaacd 100644
--- a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
+++ b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
@@ -107,7 +107,7 @@ public class ServiceConstants {
     public static final String SENTRY_STORE_GROUP_MAPPING_DEFAULT = SENTRY_STORE_HADOOP_GROUP_MAPPING;
 
     public static final String SENTRY_STORE_ORPHANED_PRIVILEGE_REMOVAL = "sentry.store.orphaned.privilege.removal";
-    public static final String SENTRY_STORE_ORPHANED_PRIVILEGE_REMOVAL_DEFAULT = "true";
+    public static final String SENTRY_STORE_ORPHANED_PRIVILEGE_REMOVAL_DEFAULT = "false";
 
     public static final ImmutableMap<String, String> SENTRY_STORE_DEFAULTS =
         ImmutableMap.<String, String>builder()

http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/3d3f96ca/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
index e8be81f..5bb00f5 100644
--- a/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
+++ b/sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
@@ -46,6 +46,7 @@ import org.apache.sentry.provider.file.PolicyFile;
 import org.apache.sentry.service.thrift.ServiceConstants.ServerConfig;
 import org.junit.After;
 import org.junit.Before;
+import org.junit.Ignore;
 import org.junit.Test;
 
 import com.google.common.collect.Iterables;
@@ -261,6 +262,7 @@ public class TestSentryStore {
    * cleanup thread runs, and expect them all to be gone from the database.
    * @throws Exception
    */
+  @Ignore("Disabled with SENTRY-545 following SENTRY-140 problems")
   @Test
   public void testPrivilegeCleanup() throws Exception {
     final String roleName = "test-priv-cleanup";
@@ -302,6 +304,7 @@ public class TestSentryStore {
    * cleanup thread.
    * @throws Exception
    */
+  @Ignore("Disabled with SENTRY-545 following SENTRY-140 problems")
   @Test
   public void testPrivilegeCleanup2() throws Exception {
     final String roleName = "test-priv-cleanup";


Mime
View raw message