This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/main by this push:
new 2e6919db6e [#8677] fix(test): Fix IT `FilesetGCSCatalogIT` fails due
to place-holder (#8678)
2e6919db6e is described below
commit 2e6919db6ed2b0b4bb88a697c64e576b64b2e2c1
Author: Mini Yu <[email protected]>
AuthorDate: Thu Sep 25 14:12:05 2025 +0800
[#8677] fix(test): Fix IT `FilesetGCSCatalogIT` fails due to place-holder
(#8678)
### What changes were proposed in this pull request?
Fix test error in `FilesetGCSCatalogIT`. The reason is that GCS path
does not support the place-holder `{{}}`
### Why are the changes needed?
It's a bug.
Fix: #8677
### Does this PR introduce _any_ user-facing change?
N/A
### How was this patch tested?
Test locally.
<img width="1604" height="532" alt="image"
src="https://github.com/user-attachments/assets/6ec67fe0-1f4c-4c39-80e2-46dd76c57d25"
/>
---
.../gravitino/catalog/fileset/integration/test/FilesetCatalogIT.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
a/catalogs/catalog-fileset/src/test/java/org/apache/gravitino/catalog/fileset/integration/test/FilesetCatalogIT.java
b/catalogs/catalog-fileset/src/test/java/org/apache/gravitino/catalog/fileset/integration/test/FilesetCatalogIT.java
index b1a9efb14b..9712e6cfa0 100644
---
a/catalogs/catalog-fileset/src/test/java/org/apache/gravitino/catalog/fileset/integration/test/FilesetCatalogIT.java
+++
b/catalogs/catalog-fileset/src/test/java/org/apache/gravitino/catalog/fileset/integration/test/FilesetCatalogIT.java
@@ -1300,7 +1300,8 @@ public class FilesetCatalogIT extends BaseIT {
Path location = new Path(storageLocation);
try {
fileSystem.deleteOnExit(location);
- } catch (IOException e) {
+ // Catch any exception to avoid test failure.
+ } catch (Exception e) {
LOG.warn("Failed to delete location: {}", location, e);
}
}