Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id 5EE06200D08 for ; Wed, 23 Aug 2017 00:22:45 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id 5D412167ACB; Tue, 22 Aug 2017 22:22:45 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 7C7E6167ACC for ; Wed, 23 Aug 2017 00:22:44 +0200 (CEST) Received: (qmail 22764 invoked by uid 500); 22 Aug 2017 22:22:43 -0000 Mailing-List: contact commits-help@sentry.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@sentry.apache.org Delivered-To: mailing list commits@sentry.apache.org Received: (qmail 22744 invoked by uid 99); 22 Aug 2017 22:22:42 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 22 Aug 2017 22:22:42 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 92DABDFF9F; Tue, 22 Aug 2017 22:22:41 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: vamsee@apache.org To: commits@sentry.apache.org Message-Id: <61c5277b950147428f708d8101ef966a@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: sentry git commit: SENTRY-1897: Rename sentry property to provide the list of sentry servers (Vamsee Yarlagadda, Reviewed by: Na Li, Alexander Kolbasov) Date: Tue, 22 Aug 2017 22:22:41 +0000 (UTC) archived-at: Tue, 22 Aug 2017 22:22:45 -0000 Repository: sentry Updated Branches: refs/heads/master 5eae90ea9 -> ebb94b189 SENTRY-1897: Rename sentry property to provide the list of sentry servers (Vamsee Yarlagadda, Reviewed by: Na Li, Alexander Kolbasov) Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/ebb94b18 Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/ebb94b18 Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/ebb94b18 Branch: refs/heads/master Commit: ebb94b1897e74fd4ce98b848845b441da6f99672 Parents: 5eae90e Author: Vamsee Yarlagadda Authored: Mon Jul 17 16:39:52 2017 -0800 Committer: Vamsee Yarlagadda Committed: Tue Aug 22 18:21:27 2017 -0400 ---------------------------------------------------------------------- conf/sentry-site.xml.hive-client.example | 2 +- conf/sentry-site.xml.hive-client.template | 2 +- .../core/common/transport/SentryClientTransportConstants.java | 4 ++-- .../src/main/java/org/apache/sentry/hdfs/ServiceConstants.java | 2 +- .../org/apache/sentry/service/thrift/ServiceConstants.java | 2 +- .../org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java | 6 +++--- .../apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java | 6 +++--- 7 files changed, 12 insertions(+), 12 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/conf/sentry-site.xml.hive-client.example ---------------------------------------------------------------------- diff --git a/conf/sentry-site.xml.hive-client.example b/conf/sentry-site.xml.hive-client.example index c9f1d05..64b1b7d 100644 --- a/conf/sentry-site.xml.hive-client.example +++ b/conf/sentry-site.xml.hive-client.example @@ -35,7 +35,7 @@ 8038 - sentry.service.client.server.rpc-address + sentry.service.client.server.rpc-addresses localhost http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/conf/sentry-site.xml.hive-client.template ---------------------------------------------------------------------- diff --git a/conf/sentry-site.xml.hive-client.template b/conf/sentry-site.xml.hive-client.template index becff9c..4703470 100644 --- a/conf/sentry-site.xml.hive-client.template +++ b/conf/sentry-site.xml.hive-client.template @@ -37,7 +37,7 @@ - sentry.service.client.server.rpc-address + sentry.service.client.server.rpc-addresses TCP address of the sentry store server http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java ---------------------------------------------------------------------- diff --git a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java b/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java index fd07887..358d282 100644 --- a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java +++ b/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/transport/SentryClientTransportConstants.java @@ -67,7 +67,7 @@ public final class SentryClientTransportConstants { static final String SERVER_RPC_PORT = "sentry.service.client.server.rpc-port"; //configuration for server address. It can be coma seperated list of server addresses. - static final String SERVER_RPC_ADDRESS = "sentry.service.client.server.rpc-address"; + static final String SERVER_RPC_ADDRESS = "sentry.service.client.server.rpc-addresses"; /** * This configuration parameter is only meant to be used for testing purposes. @@ -140,7 +140,7 @@ public final class SentryClientTransportConstants { static final String SERVER_RPC_PORT = "sentry.hdfs.service.client.server.rpc-port"; //configuration for server address. It can be coma seperated list of server addresses. - static final String SERVER_RPC_ADDRESS = "sentry.hdfs.service.client.server.rpc-address"; + static final String SERVER_RPC_ADDRESS = "sentry.hdfs.service.client.server.rpc-addresses"; /** * This configuration parameter is only meant to be used for testing purposes. http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java ---------------------------------------------------------------------- diff --git a/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java b/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java index f993010..55092ee 100644 --- a/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java +++ b/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/ServiceConstants.java @@ -73,7 +73,7 @@ public class ServiceConstants { public static final String SERVER_RPC_PORT = "sentry.hdfs.service.client.server.rpc-port"; - public static final String SERVER_RPC_ADDRESS = "sentry.hdfs.service.client.server.rpc-address"; + public static final String SERVER_RPC_ADDRESS = "sentry.hdfs.service.client.server.rpc-addresses"; public static final String USE_COMPACT_TRANSPORT = "sentry.hdfs.service.client.compact.transport"; public static final boolean USE_COMPACT_TRANSPORT_DEFAULT = false; http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/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 5b6ef53..48aec1e 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 @@ -241,7 +241,7 @@ public class ServiceConstants { public static class ClientConfig { public static final String SERVER_RPC_PORT = "sentry.service.client.server.rpc-port"; public static final int SERVER_RPC_PORT_DEFAULT = ServerConfig.RPC_PORT_DEFAULT; - public static final String SERVER_RPC_ADDRESS = "sentry.service.client.server.rpc-address"; + public static final String SERVER_RPC_ADDRESS = "sentry.service.client.server.rpc-addresses"; public static final String SERVER_RPC_CONN_TIMEOUT = "sentry.service.client.server.rpc-connection-timeout"; // HA configuration http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java ---------------------------------------------------------------------- diff --git a/sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java b/sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java index c19ccbd..acf1cf5 100644 --- a/sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java +++ b/sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java @@ -329,7 +329,7 @@ public class TestHDFSIntegration { hadoopConf.set("sentry.authorization-provider.cache-stale-threshold.ms", String.valueOf(STALE_THRESHOLD)); hadoopConf.set("sentry.hdfs.service.security.mode", "none"); - hadoopConf.set("sentry.hdfs.service.client.server.rpc-address", "localhost"); + hadoopConf.set("sentry.hdfs.service.client.server.rpc-addresses", "localhost"); hadoopConf.set("sentry.hdfs.service.client.server.rpc-port", String.valueOf(sentryPort)); EditLogFileOutputStream.setShouldSkipFsyncForTesting(true); miniDFS = new MiniDFSCluster.Builder(hadoopConf).build(); @@ -487,8 +487,8 @@ public class TestHDFSIntegration { public Void run() throws Exception { HiveConf hiveConf = hiveConfiguration; hiveConf.set("sentry.metastore.plugins", "org.apache.sentry.hdfs.MetastorePlugin"); - hiveConf.set("sentry.service.client.server.rpc-address", "localhost"); - hiveConf.set("sentry.hdfs.service.client.server.rpc-address", "localhost"); + hiveConf.set("sentry.service.client.server.rpc-addresses", "localhost"); + hiveConf.set("sentry.hdfs.service.client.server.rpc-addresses", "localhost"); hiveConf.set("sentry.hdfs.service.client.server.rpc-port", String.valueOf(sentryPort)); hiveConf.set("sentry.service.client.server.rpc-port", String.valueOf(sentryPort)); // hiveConf.set("sentry.service.server.compact.transport", "true"); http://git-wip-us.apache.org/repos/asf/sentry/blob/ebb94b18/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java ---------------------------------------------------------------------- diff --git a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java index e62c54a..63e718c 100644 --- a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java +++ b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java @@ -526,8 +526,8 @@ public abstract class TestHDFSIntegrationBase { public Void run() throws Exception { HiveConf hiveConf = hiveConfiguration; hiveConf.set("sentry.metastore.plugins", "org.apache.sentry.hdfs.MetastorePlugin"); - hiveConf.set("sentry.service.client.server.rpc-address", "localhost"); - hiveConf.set("sentry.hdfs.service.client.server.rpc-address", "localhost"); + hiveConf.set("sentry.service.client.server.rpc-addresses", "localhost"); + hiveConf.set("sentry.hdfs.service.client.server.rpc-addresses", "localhost"); hiveConf.set("sentry.hdfs.service.client.server.rpc-port", String.valueOf(sentryPort)); hiveConf.set("sentry.service.client.server.rpc-port", String.valueOf(sentryPort)); // hiveConf.set("sentry.service.server.compact.transport", "true"); @@ -694,7 +694,7 @@ public abstract class TestHDFSIntegrationBase { hadoopConf.set("sentry.authorization-provider.cache-stale-threshold.ms", String.valueOf(STALE_THRESHOLD)); hadoopConf.set("sentry.hdfs.service.security.mode", "none"); - hadoopConf.set("sentry.hdfs.service.client.server.rpc-address", "localhost"); + hadoopConf.set("sentry.hdfs.service.client.server.rpc-addresses", "localhost"); hadoopConf.set("sentry.hdfs.service.client.server.rpc-port", String.valueOf(sentryPort)); EditLogFileOutputStream.setShouldSkipFsyncForTesting(true); miniDFS = new MiniDFSCluster.Builder(hadoopConf).build();