Revanth14 opened a new pull request, #1285:
URL: https://github.com/apache/iceberg-go/pull/1285

   ## Summary
   
   Adds unit coverage for `PurgeTable` in the Glue and Hive catalogs.
   
   The new tests verify that:
   
   - `PurgeTable` loads the table and drops the catalog entry
   - local table metadata/data files are physically deleted
   - `PurgeFiles` failures are swallowed after the drop
   - the warning path is logged when file purge fails
   
   ## Why
   
   `glue.PurgeTable` and `hive.PurgeTable` already call `DropTable` followed by 
best-effort `tbl.PurgeFiles`, but only `DropTable` had test coverage. These 
tests lock in the intended purge behavior and the existing swallow-and-log 
error handling.
   
   Fixes #1272.
   
   ## Testing
   
   ```sh
   go test ./catalog/glue ./catalog/hive
   go test ./...


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to