Fix findbugs issue

apache · Jan 3, 2025 · 6621014 · 6621014
1 parent aa2cbc1
commit 6621014
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/.../src/main/java/org/apache/uniffle/coordinator/access/checker/AccessSupportRssChecker.java b/.../src/main/java/org/apache/uniffle/coordinator/access/checker/AccessSupportRssChecker.java
@@ -45,7 +45,7 @@ public AccessSupportRssChecker(AccessManager accessManager) throws Exception {
     super(accessManager);
     List<String> unsupportedConfigs =
         accessManager.getCoordinatorConf().get(CoordinatorConf.COORDINATOR_UNSUPPORTED_CONFIGS);
-    unsupportedConfigMap = new HashMap<>(unsupportedConfigs.size());
+    unsupportedConfigMap = new HashMap<>();
     if (unsupportedConfigs != null && !unsupportedConfigs.isEmpty()) {
       for (String keyValue : unsupportedConfigs) {
         String[] pair = keyValue.split(":", 2);
@@ -60,9 +60,6 @@ public AccessSupportRssChecker(AccessManager accessManager) throws Exception {
 
   @Override
   public AccessCheckResult check(AccessInfo accessInfo) {
-    if (unsupportedConfigMap == null || unsupportedConfigMap.isEmpty()) {
-      return new AccessCheckResult(true, Constants.COMMON_SUCCESS_MESSAGE);
-    }
     for (Map.Entry<String, String> entry : unsupportedConfigMap.entrySet()) {
       String unsupportedConfKey = entry.getKey();
       String unsupportedConfValue = entry.getValue();