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 <79496857+hmangl...@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.