This is an automated email from the ASF dual-hosted git repository. github-actions[bot] pushed a commit to branch cherry-pick-d1e73647-to-branch-1.3 in repository https://gitbox.apache.org/repos/asf/gravitino.git
commit f014f0c8e2cbcae946d0c791c469055bdfe998da Author: Qi Yu <[email protected]> AuthorDate: Fri Sep 11 12:04:28 2026 +0800 [MINOR] test(lance): Fix missing-schema assertion for Lance Spark 0.5.1 (#13084) ### What changes were proposed in this pull request? Make the missing-schema assertion in `LanceSparkRESTServiceIT` compatible with Lance Spark 0.5.1 by removing the requirement that its error message contain `NoSuchSchemaException`. Keep checks that table creation throws, the message identifies the missing schema and says it does not exist, and the schema remains absent. ### Why are the changes needed? Lance Spark 0.5.1 reports `Namespace not found: Schema ... does not exist` without the Gravitino exception class name. The existing assertion fails despite the expected missing-schema behavior, breaking the [compatibility matrix](https://github.com/apache/gravitino/actions/runs/34051320348/job/101535350951). ### Does this PR introduce _any_ user-facing change? No. Test-only change. ### How was this patch tested? Reproduced the original failure locally on the CI commit: 0.2.0 and 0.4.0 passed all 12 tests each; 0.5.1 failed only this assertion. With the change, all 36 tests passed across the three versions. Revalidated on the latest main with formatting: ```bash SKIP_DOCKER_TESTS=true ./gradlew spotlessApply \ :lance:lance-rest-server:lanceSparkMatrixTest \ -PlanceSparkBundleVersions=0.2.0,0.4.0,0.5.1 \ -PskipDockerTests=true -PskipWeb=true ``` --- .../gravitino/lance/integration/test/LanceSparkRESTServiceIT.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lance/lance-rest-server/src/test/java/org/apache/gravitino/lance/integration/test/LanceSparkRESTServiceIT.java b/lance/lance-rest-server/src/test/java/org/apache/gravitino/lance/integration/test/LanceSparkRESTServiceIT.java index 670f547688..d680c290c7 100644 --- a/lance/lance-rest-server/src/test/java/org/apache/gravitino/lance/integration/test/LanceSparkRESTServiceIT.java +++ b/lance/lance-rest-server/src/test/java/org/apache/gravitino/lance/integration/test/LanceSparkRESTServiceIT.java @@ -299,8 +299,7 @@ public class LanceSparkRESTServiceIT extends BaseIT { RuntimeException exception = Assertions.assertThrows( RuntimeException.class, () -> createLanceTable(nonExistentSchemaName, tableName)); - assertFailureContainsAll( - exception, "NoSuchSchemaException", nonExistentSchemaName, "does not exist"); + assertFailureContainsAll(exception, nonExistentSchemaName, "does not exist"); Assertions.assertFalse(catalog.asSchemas().schemaExists(nonExistentSchemaName)); }
