yuqi1129 opened a new pull request, #9967: URL: https://github.com/apache/gravitino/pull/9967
### What changes were proposed in this pull request? This pull request improves the robustness of table purge and drop operations in the Lance catalog by handling cases where the underlying data location is missing or was never created. It also adds comprehensive integration tests to verify these scenarios and updates the REST service integration tests to align with the new behavior. **Enhancements to Table Purge/Drop Logic:** * Updated `LanceTableOperations.purgeTable` to check if the dataset location exists before attempting deletion, preventing errors when the data is already missing or was never created. The method now logs a warning and skips deletion if the dataset cannot be opened. * Added import for `ReadOptions` in `LanceTableOperations` to support the new dataset existence check. **Integration and REST API Test Improvements:** * Added new integration tests in `CatalogGenericCatalogLanceIT` to verify that purging tables works correctly when the data location is missing, never created, or for managed tables with auto-generated locations. * Updated REST service integration test `LanceRESTServiceIT` to assert that dropping a table does not throw an exception when the location is missing, and added a new test to cover dropping tables with missing or never-created locations. ### Why are the changes needed? It's a bug. Fix: #9954 ### Does this PR introduce _any_ user-facing change? N/A. ### How was this patch tested? ITs. -- 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]
