From c79d3d07844385a3484c03e7dd34a8c6630145c3 Mon Sep 17 00:00:00 2001 From: Kai Huang Date: Thu, 7 May 2026 09:52:58 -0700 Subject: [PATCH] Drop @Ignore'd Calcite ITs from CalciteNoPushdownIT @Suite MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Mirrors the trailing portion of opensearch-project/sql#5414 onto feature/mustang-ppl-integration. The Gradle 9.4.1 wrapper bump (#5406) and the integTest exclude block for @Ignore'd classes (#5407) already landed on this branch, but the matching cleanup of `CalciteNoPushdownIT` did not — the @Suite still listed four classes that are now build-time excluded from integTest: - CalciteInformationSchemaCommandIT - CalciteJsonFunctionsIT - CalcitePrometheusDataSourceCommandsIT - CalciteShowDataSourcesCommandIT Leaving them in the suite means the JUnit runner attempts to load each class, finds it referenced from the suite, and registers a descriptor — which (combined with the class-level @Ignore) reproduces the same TestEventReporterAsListener.started cast that the integTest exclude was meant to avoid. Dropping the entries here keeps the no-pushdown suite consistent with the exclude block: a class excluded at the build layer must not also appear in any @Suite.SuiteClasses list. No new tests are introduced or removed — the four classes are already @Ignore'd and have been since they were written. This change is purely about keeping the no-pushdown suite metadata aligned with the exclusion pattern carried over from main. Signed-off-by: Kai Huang --- .../java/org/opensearch/sql/calcite/CalciteNoPushdownIT.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/integ-test/src/test/java/org/opensearch/sql/calcite/CalciteNoPushdownIT.java b/integ-test/src/test/java/org/opensearch/sql/calcite/CalciteNoPushdownIT.java index a56715845ab..3a096ec9617 100644 --- a/integ-test/src/test/java/org/opensearch/sql/calcite/CalciteNoPushdownIT.java +++ b/integ-test/src/test/java/org/opensearch/sql/calcite/CalciteNoPushdownIT.java @@ -43,10 +43,8 @@ CalciteGeoIpFunctionsIT.class, CalciteGeoPointFormatsIT.class, CalciteHeadCommandIT.class, - CalciteInformationSchemaCommandIT.class, CalciteIPComparisonIT.class, CalciteIPFunctionsIT.class, - CalciteJsonFunctionsIT.class, CalciteLegacyAPICompatibilityIT.class, CalciteLikeQueryIT.class, CalciteMathematicalFunctionIT.class, @@ -88,7 +86,6 @@ CalcitePPLSortIT.class, CalcitePPLStringBuiltinFunctionIT.class, CalcitePPLTrendlineIT.class, - CalcitePrometheusDataSourceCommandsIT.class, CalciteQueryAnalysisIT.class, CalciteRareCommandIT.class, CalciteRegexCommandIT.class, @@ -99,7 +96,6 @@ CalciteResourceMonitorIT.class, CalciteSearchCommandIT.class, CalciteSettingsIT.class, - CalciteShowDataSourcesCommandIT.class, CalciteSortCommandIT.class, CalciteStatsCommandIT.class, CalciteSystemFunctionIT.class,