Hi all, I'd appreciate a review of https://github.com/apache/iceberg/pull/18463.
GCSFileIO.deleteFiles currently can't tell a missing object from a failed delete, because Storage.delete(Iterable<BlobId>) returns false for both. Real failures, such as permission errors, are silently swallowed, so callers have nothing to retry. The PR switches deleteFiles to StorageBatch, which reports each object's outcome separately. - A missing object counts as success, so retries are safe to repeat. - Any real failure is raised as BulkDeletionFailureException, so the caller knows what failed and can retry it. - If the batch request itself fails, every object in that batch is reported as failed. Regards, Munendra S N
