yuqi1129 opened a new pull request, #13084:
URL: https://github.com/apache/gravitino/pull/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
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to