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 fabed9cf3 [MINOR] Remove unused throw exception in 
SecureHadoopCatalogOperations#testConnection (#5607)
fabed9cf3 is described below

commit fabed9cf3480380a9e9c4df579b3f5130c270f74
Author: Ken Huang <[email protected]>
AuthorDate: Tue Nov 19 15:06:58 2024 +0800

    [MINOR] Remove unused throw exception in 
SecureHadoopCatalogOperations#testConnection (#5607)
    
    ### What changes were proposed in this pull request?
    
    the `testConnection` throw Exception is redundant in this method, so we
    should remove it.
    
    ### Why are the changes needed?
    update the code quality
    
    ### Does this PR introduce _any_ user-facing change?
    n/a
    
    ### How was this patch tested?
    n/a
---
 .../apache/gravitino/catalog/hadoop/SecureHadoopCatalogOperations.java | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git 
a/catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/SecureHadoopCatalogOperations.java
 
b/catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/SecureHadoopCatalogOperations.java
index b0717a90c..9d21a1782 100644
--- 
a/catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/SecureHadoopCatalogOperations.java
+++ 
b/catalogs/catalog-hadoop/src/main/java/org/apache/gravitino/catalog/hadoop/SecureHadoopCatalogOperations.java
@@ -243,8 +243,7 @@ public class SecureHadoopCatalogOperations
       Catalog.Type type,
       String provider,
       String comment,
-      Map<String, String> properties)
-      throws Exception {
+      Map<String, String> properties) {
     hadoopCatalogOperations.testConnection(catalogIdent, type, provider, 
comment, properties);
   }
 

Reply via email to