YARN-11180. Fix CheckStyle.

apache · Jul 27, 2022 · 515f422 · 515f422
1 parent 730ea24
commit 515f422
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/.../main/java/org/apache/hadoop/yarn/server/router/clientrm/FederationClientInterceptor.java b/.../main/java/org/apache/hadoop/yarn/server/router/clientrm/FederationClientInterceptor.java
@@ -250,7 +250,7 @@ protected ApplicationClientProtocol getClientRMProxyForSubCluster(
 
   private SubClusterId getRandomActiveSubCluster(
       Map<SubClusterId, SubClusterInfo> activeSubClusters) throws YarnException {
-    if (activeSubClusters != null && activeSubClusters.isEmpty()) {
+    if (activeSubClusters == null || activeSubClusters.isEmpty()) {
       RouterServerUtil.logAndThrowException(
           FederationPolicyUtils.NO_ACTIVE_SUBCLUSTER_AVAILABLE, null);
     }

diff --git a/...t/java/org/apache/hadoop/yarn/server/router/clientrm/TestFederationClientInterceptor.java b/...t/java/org/apache/hadoop/yarn/server/router/clientrm/TestFederationClientInterceptor.java
@@ -354,7 +354,7 @@ public void testForceKillApplicationNotExists() throws Exception {
 
     LambdaTestUtils.intercept(YarnException.class,
         "Application " + appId + " does not exist in FederationStateStore.",
-         () -> interceptor.forceKillApplication(requestKill));
+        () -> interceptor.forceKillApplication(requestKill));
   }
 
   /**