bbiiaaoo opened a new pull request, #11626:
URL: https://github.com/apache/gravitino/pull/11626

   ### What changes were proposed in this pull request?
   
   This PR fixes the Lance REST per-namespace `ListTables` implementation to 
return child table names instead of fully-qualified `catalog.schema.table` 
strings.
   
   It also tightens the related REST and Spark integration tests:
   - `LanceRESTServiceIT` now expects `ListTables` to return only the leaf 
table name.
   - `LanceSparkRESTServiceIT` now validates the raw `SHOW TABLES` table name 
instead of stripping namespace prefixes.
   
   ### Why are the changes needed?
   
   The Lance Namespace `ListTables` API lists child table names under the 
requested parent namespace. The current implementation returns fully-qualified 
names, causing Spark `SHOW TABLES` to display polluted table names such as 
`catalog.schema.table` instead of `table`.
   
   Fix: #11588
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes. The Lance REST per-namespace `ListTables` endpoint now returns leaf 
table names in the `tables` field.
   
   The REST response shape is unchanged.
   
   ### How was this patch tested?
   
   Ran:
   
   ```bash
   ./gradlew :lance:lance-rest-server:testClasses
   ./gradlew :lance:lance-rest-server:test --tests 
"org.apache.gravitino.lance.integration.test.LanceRESTServiceIT" 
-PskipDockerTests=false
   ./gradlew :lance:lance-rest-server:lanceSparkMatrixTest 
-PskipDockerTests=false


-- 
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