diff --git a/pinot-clients/pinot-java-client/pom.xml b/pinot-clients/pinot-java-client/pom.xml index 77ba9e73eed..df86a9f4fb4 100644 --- a/pinot-clients/pinot-java-client/pom.xml +++ b/pinot-clients/pinot-java-client/pom.xml @@ -35,16 +35,6 @@ ${basedir}/../.. - - - org.apache.maven.plugins - maven-surefire-plugin - - 1 - true - - - src/main/resources diff --git a/pinot-clients/pinot-jdbc-client/pom.xml b/pinot-clients/pinot-jdbc-client/pom.xml index dae6a3b335f..0ceded3e924 100644 --- a/pinot-clients/pinot-jdbc-client/pom.xml +++ b/pinot-clients/pinot-jdbc-client/pom.xml @@ -35,16 +35,6 @@ ${basedir}/../.. - - - org.apache.maven.plugins - maven-surefire-plugin - - 1 - true - - - src/main/resources diff --git a/pinot-common/pom.xml b/pinot-common/pom.xml index 339d8f55d2b..9fedfd444a4 100644 --- a/pinot-common/pom.xml +++ b/pinot-common/pom.xml @@ -44,6 +44,18 @@ + + org.apache.maven.plugins + maven-surefire-plugin + + + + usedefaultlisteners + false + + + + org.antlr antlr4-maven-plugin diff --git a/pinot-connectors/pinot-spark-2-connector/pom.xml b/pinot-connectors/pinot-spark-2-connector/pom.xml index eaae57887c2..c92e564c7fc 100644 --- a/pinot-connectors/pinot-spark-2-connector/pom.xml +++ b/pinot-connectors/pinot-spark-2-connector/pom.xml @@ -231,7 +231,6 @@ org.apache.maven.plugins maven-surefire-plugin - 2.7 true diff --git a/pinot-connectors/pinot-spark-3-connector/pom.xml b/pinot-connectors/pinot-spark-3-connector/pom.xml index fe5515ab4ad..1bdb23edbdc 100644 --- a/pinot-connectors/pinot-spark-3-connector/pom.xml +++ b/pinot-connectors/pinot-spark-3-connector/pom.xml @@ -228,7 +228,6 @@ org.apache.maven.plugins maven-surefire-plugin - 2.7 true diff --git a/pinot-connectors/pinot-spark-common/pom.xml b/pinot-connectors/pinot-spark-common/pom.xml index 427e23d9240..bfbe8c3f1ee 100644 --- a/pinot-connectors/pinot-spark-common/pom.xml +++ b/pinot-connectors/pinot-spark-common/pom.xml @@ -246,7 +246,6 @@ org.apache.maven.plugins maven-surefire-plugin - 2.7 true diff --git a/pinot-controller/pom.xml b/pinot-controller/pom.xml index 0365d1e58d1..3607c88ace6 100644 --- a/pinot-controller/pom.xml +++ b/pinot-controller/pom.xml @@ -246,11 +246,6 @@ org.apache.maven.plugins maven-surefire-plugin - - 3.0.0-M5 testng-statefull.xml diff --git a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java index c29fbe59f17..dcec37ceefb 100644 --- a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java +++ b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java @@ -209,7 +209,7 @@ public void setUp() // - Failed to copy segment tar file to final location due to the same segment pushed twice concurrently. // In such cases we upload all the segments again to ensure that the data is setup correctly. assertTrue(e.getMessage().contains("Another segment upload is in progress for segment") || e.getMessage() - .contains("Failed to update ZK metadata for segment") || e.getMessage() + .contains("Failed to create ZK metadata for segment") || e.getMessage() .contains("java.nio.file.FileAlreadyExistsException"), e.getMessage()); uploadSegments(getTableName(), _tarDir); } @@ -238,7 +238,7 @@ public void setUp() // If enableParallelPushProtection is enabled and the same segment is uploaded concurrently, we could get one // of the three exception: // - 409 conflict of the second call enters ProcessExistingSegment ; - // - segmentZkMetadata creation failure if both calls entered ProcessNewSegment. + // - segmentZkMetadata update failure if both calls entered ProcessNewSegment. // - Failed to copy segment tar file to final location due to the same segment pushed twice concurrently. // In such cases we upload all the segments again to ensure that the data is setup correctly. assertTrue(e.getMessage().contains("Another segment upload is in progress for segment") || e.getMessage() @@ -1133,6 +1133,7 @@ public void testRangeIndexTriggering(boolean useMultiStageQueryEngine) // Update table config and trigger reload TableConfig tableConfig = getOfflineTableConfig(); tableConfig.getIndexingConfig().setRangeIndexColumns(UPDATED_RANGE_INDEX_COLUMNS); + updateTableConfig(tableConfig); reloadAllSegments(TEST_UPDATED_RANGE_INDEX_QUERY, false, numTotalDocs); assertEquals(postQuery(TEST_UPDATED_RANGE_INDEX_QUERY).get("numEntriesScannedInFilter").asLong(), 0L); @@ -1140,7 +1141,7 @@ public void testRangeIndexTriggering(boolean useMultiStageQueryEngine) tableConfig = getOfflineTableConfig(); tableConfig.getIndexingConfig().setRangeIndexColumns(getRangeIndexColumns()); updateTableConfig(tableConfig); - reloadAllSegments(TEST_UPDATED_RANGE_INDEX_QUERY, false, numTotalDocs); + reloadAllSegments(TEST_UPDATED_RANGE_INDEX_QUERY, true, numTotalDocs); assertEquals(postQuery(TEST_UPDATED_RANGE_INDEX_QUERY).get("numEntriesScannedInFilter").asLong(), numTotalDocs); assertEquals(getTableSize(getTableName()), _tableSizeAfterRemovingIndex); } @@ -1163,7 +1164,7 @@ public void testBloomFilterTriggering() tableConfig = getOfflineTableConfig(); tableConfig.getIndexingConfig().setBloomFilterColumns(getBloomFilterColumns()); updateTableConfig(tableConfig); - reloadAllSegments(TEST_UPDATED_BLOOM_FILTER_QUERY, false, numTotalDocs); + reloadAllSegments(TEST_UPDATED_BLOOM_FILTER_QUERY, true, numTotalDocs); assertEquals(postQuery(TEST_UPDATED_BLOOM_FILTER_QUERY).get("numSegmentsProcessed").asLong(), NUM_SEGMENTS); assertEquals(getTableSize(getTableName()), _tableSizeAfterRemovingIndex); } diff --git a/pom.xml b/pom.xml index 332dee84973..8d540dc4b4b 100644 --- a/pom.xml +++ b/pom.xml @@ -162,6 +162,8 @@ 4.1 2.20.94 2.12.5 + 7.8.0 + -Xms4g -Xmx4g 3.22.0 @@ -174,9 +176,9 @@ org.apache.pinot.shaded - 3.0.0-M5 3.3.0 3.4.1 + 3.1.2 @@ -399,7 +401,7 @@ org.testng testng - 6.11 + ${testng.version} org.roaringbitmap