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 <[email protected]>
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.
