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