hive git commit: HIVE-18147 : Tests can fail with java.net.BindException: Address already in use (Janaki Lahorani, reviewed by Andrew Sherman and Vihang Karajgaonkar)

2017-12-03 Thread vihangk1
Repository: hive
Updated Branches:
  refs/heads/branch-2 059844e48 -> 1f7e3be05


HIVE-18147 : Tests can fail with java.net.BindException: Address already in use 
(Janaki Lahorani, reviewed by Andrew Sherman and Vihang Karajgaonkar)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/1f7e3be0
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/1f7e3be0
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/1f7e3be0

Branch: refs/heads/branch-2
Commit: 1f7e3be05982a4cb41f9316bf2c012767b5b2a5f
Parents: 059844e
Author: Vihang Karajgaonkar 
Authored: Sun Dec 3 15:36:15 2017 +0530
Committer: Vihang Karajgaonkar 
Committed: Sun Dec 3 15:43:15 2017 +0530

--
 .../apache/hive/hcatalog/cli/TestPermsGrp.java  |  4 +-
 .../mapreduce/TestHCatPartitionPublish.java |  4 +-
 .../hive/hcatalog/templeton/TestWebHCatE2e.java | 41 +---
 .../AbstractTestAuthorizationApiAuthorizer.java |  3 +-
 .../hadoop/hive/metastore/TestFilterHooks.java  |  3 +-
 .../metastore/TestHiveMetaStoreStatsMerge.java  |  3 +-
 ...TestHiveMetaStoreWithEnvironmentContext.java |  3 +-
 .../TestMetaStoreEndFunctionListener.java   |  3 +-
 .../metastore/TestMetaStoreEventListener.java   |  3 +-
 .../TestMetaStoreEventListenerOnlyOnCommit.java |  3 +-
 .../metastore/TestMetaStoreInitListener.java|  3 +-
 .../hive/metastore/TestMetaStoreMetrics.java|  7 ++--
 .../hive/metastore/TestRemoteHiveMetaStore.java |  3 +-
 .../TestRemoteHiveMetaStoreIpAddress.java   |  5 +--
 .../hive/metastore/TestRetryingHMSHandler.java  |  3 +-
 ...estDDLWithRemoteMetastoreSecondNamenode.java |  3 +-
 .../hive/ql/security/FolderPermissionBase.java  |  3 +-
 .../security/StorageBasedMetastoreTestBase.java |  5 +--
 .../TestAuthorizationPreEventListener.java  |  4 +-
 .../TestClientSideAuthorizationProvider.java|  4 +-
 .../TestMetastoreAuthorizationProvider.java |  5 +--
 .../TestMultiAuthorizationPreEventListener.java |  6 +--
 .../thrift/ThriftCliServiceMessageSizeTest.java | 24 +---
 .../org/apache/hive/jdbc/miniHS2/MiniHS2.java   | 37 +++---
 .../hadoop/hive/metastore/MetaStoreUtils.java   | 33 
 .../thrift/ThriftCliServiceTestWithCookie.java  | 25 +---
 26 files changed, 155 insertions(+), 85 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/hive/blob/1f7e3be0/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
--
diff --git 
a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java 
b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
index 66a5dd4..2375f5d 100644
--- a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
+++ b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
@@ -79,9 +79,7 @@ public class TestPermsGrp extends TestCase {
   return;
 }
 
-
-msPort = MetaStoreUtils.findFreePort();
-MetaStoreUtils.startMetaStore(msPort, 
ShimLoader.getHadoopThriftAuthBridge());
+msPort = MetaStoreUtils.startMetaStoreWithRetry();
 
 isServerRunning = true;
 

http://git-wip-us.apache.org/repos/asf/hive/blob/1f7e3be0/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
--
diff --git 
a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
 
b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
index 69874bc..777fc9c 100644
--- 
a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
+++ 
b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
@@ -104,10 +104,8 @@ public class TestHCatPartitionPublish {
   return;
 }
 
-msPort = MetaStoreUtils.findFreePort();
+msPort = MetaStoreUtils.startMetaStoreWithRetry();
 
-MetaStoreUtils.startMetaStore(msPort, ShimLoader
-.getHadoopThriftAuthBridge());
 Thread.sleep(1);
 isServerRunning = true;
 securityManager = System.getSecurityManager();

http://git-wip-us.apache.org/repos/asf/hive/blob/1f7e3be0/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
--
diff --git 
a/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
 
b/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
index b62716e..841e2ff 100644
--- 
a/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
+++ 

hive git commit: HIVE-18147 : Tests can fail with java.net.BindException: Address already in use (Janaki Lahorani, reviewed by Andrew Sherman and Vihang Karajgaonkar)

2017-11-28 Thread vihangk1
Repository: hive
Updated Branches:
  refs/heads/master 525cfa7be -> 542b165c3


HIVE-18147 : Tests can fail with java.net.BindException: Address already in use 
(Janaki Lahorani, reviewed by Andrew Sherman and Vihang Karajgaonkar)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/542b165c
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/542b165c
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/542b165c

Branch: refs/heads/master
Commit: 542b165c3f042f9d726d2ed831adfc5a89ce369f
Parents: 525cfa7
Author: Vihang Karajgaonkar 
Authored: Tue Nov 28 11:11:39 2017 -0800
Committer: Vihang Karajgaonkar 
Committed: Tue Nov 28 11:30:57 2017 -0800

--
 .../apache/hive/hcatalog/cli/TestPermsGrp.java  |  4 +-
 .../mapreduce/TestHCatPartitionPublish.java |  3 +-
 .../hive/hcatalog/templeton/TestWebHCatE2e.java | 43 ++--
 .../AbstractTestAuthorizationApiAuthorizer.java |  3 +-
 .../hadoop/hive/metastore/TestFilterHooks.java  |  3 +-
 ...TestHiveMetaStoreWithEnvironmentContext.java |  3 +-
 .../TestMetaStoreEndFunctionListener.java   |  3 +-
 .../metastore/TestMetaStoreEventListener.java   |  3 +-
 .../TestMetaStoreEventListenerOnlyOnCommit.java |  3 +-
 .../metastore/TestMetaStoreInitListener.java|  3 +-
 .../hive/metastore/TestMetaStoreMetrics.java|  7 ++--
 .../hive/metastore/TestRemoteHiveMetaStore.java |  3 +-
 .../TestRemoteHiveMetaStoreIpAddress.java   |  5 +--
 .../hive/metastore/TestRetryingHMSHandler.java  |  3 +-
 ...estDDLWithRemoteMetastoreSecondNamenode.java |  3 +-
 .../security/StorageBasedMetastoreTestBase.java |  5 +--
 .../TestAuthorizationPreEventListener.java  |  4 +-
 .../TestClientSideAuthorizationProvider.java|  4 +-
 .../TestMetastoreAuthorizationProvider.java |  5 +--
 .../TestMultiAuthorizationPreEventListener.java |  6 +--
 .../org/apache/hive/jdbc/miniHS2/MiniHS2.java   | 37 ++---
 .../hive/metastore/MetaStoreTestUtils.java  | 32 +++
 .../thrift/ThriftCliServiceTestWithCookie.java  | 26 
 23 files changed, 134 insertions(+), 77 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/hive/blob/542b165c/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
--
diff --git 
a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java 
b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
index 80c5d63..e5aa2b8 100644
--- a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
+++ b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/cli/TestPermsGrp.java
@@ -80,9 +80,7 @@ public class TestPermsGrp extends TestCase {
   return;
 }
 
-
-msPort = MetaStoreTestUtils.findFreePort();
-MetaStoreTestUtils.startMetaStore(msPort, 
HadoopThriftAuthBridge.getBridge());
+msPort = MetaStoreTestUtils.startMetaStoreWithRetry();
 
 isServerRunning = true;
 

http://git-wip-us.apache.org/repos/asf/hive/blob/542b165c/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
--
diff --git 
a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
 
b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
index a000642..6711a3e 100644
--- 
a/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
+++ 
b/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java
@@ -105,9 +105,8 @@ public class TestHCatPartitionPublish {
   return;
 }
 
-msPort = MetaStoreTestUtils.findFreePort();
+msPort = MetaStoreTestUtils.startMetaStoreWithRetry();
 
-MetaStoreTestUtils.startMetaStore(msPort, 
HadoopThriftAuthBridge.getBridge());
 Thread.sleep(1);
 isServerRunning = true;
 securityManager = System.getSecurityManager();

http://git-wip-us.apache.org/repos/asf/hive/blob/542b165c/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
--
diff --git 
a/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
 
b/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
index 341314b..55cd5b1 100644
--- 
a/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
+++ 
b/hcatalog/webhcat/svr/src/test/java/org/apache/hive/hcatalog/templeton/TestWebHCatE2e.java
@@ -70,21 +70,38 @@ public class TestWebHCatE2e {
   private static final String charSet =