[hive] branch master updated: HIVE-25475: TestStatsReplicationScenarios.testForParallelBootstrapLoad is unstable (Haymant Mangla, reviewed by Mahesh Kumar Behera)

[ma...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5c53f92  HIVE-25475: TestStatsReplicationScenarios.testForParallelBootstrapLoad is unstable (Haymant Mangla, reviewed by Mahesh Kumar Behera)
5c53f92 is described below

commit 5c53f92bdf3834988db0e0c92d7eff9891bdf639
Author: Haymant Mangla <79...@users.noreply.github.com>
AuthorDate: Thu Aug 26 20:35:31 2021 +0530

    HIVE-25475: TestStatsReplicationScenarios.testForParallelBootstrapLoad is unstable (Haymant Mangla, reviewed by Mahesh Kumar Behera)
---
 .../org/apache/hadoop/hive/ql/parse/TestStatsReplicationScenarios.java  | 1 -
 ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java                | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestStatsReplicationScenarios.java b/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestStatsReplicationScenarios.java
index 62792be..cca82a8 100644
--- a/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestStatsReplicationScenarios.java
+++ b/itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/parse/TestStatsReplicationScenarios.java
@@ -684,7 +684,6 @@ public class TestStatsReplicationScenarios {
   }
 
   @Test
-  @Ignore("HIVE-25475")
   public void testForParallelBootstrapLoad() throws Throwable {
     LOG.info("Testing " + testName.getClass().getName() + "." + testName.getMethodName());
     testStatsReplicationCommon(true, false, false);
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java b/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java
index 4493928..3de2e8d 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java
@@ -5008,7 +5008,7 @@ private void constructOneLBLocationMap(FileStatus fSta,
 
     List<Path> newFiles = null;
     if (newFilesStatus != null) {
-      newFiles = new ArrayList<>();
+      newFiles = Collections.synchronizedList(new ArrayList<Path>());
     }
 
     // If we're moving files around for an ACID write then the rules and paths are all different.