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);
       }
     }

Reply via email to