Remove unneeded check

adminClient doeesn't change in the try block so no check needed in the finally block.
pbailis · Jun 20, 2013 · c809fba · c809fba
1 parent 7daac20
commit c809fba
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/...lder/src/java/voldemort/store/readonly/mr/azkaban/VoldemortMultiStoreBuildAndPushJob.java b/...lder/src/java/voldemort/store/readonly/mr/azkaban/VoldemortMultiStoreBuildAndPushJob.java
@@ -539,9 +539,8 @@ public List<String> call() throws Exception {
 
                     }
                 } finally {
-                    if(adminClient != null) {
-                        adminClient.close();
-                    }
+                    adminClient.close();
+
                 }
             }
         } catch(Exception e) {
@@ -575,9 +574,7 @@ public List<String> call() throws Exception {
 
                     }
                 } finally {
-                    if(adminClient != null) {
-                        adminClient.close();
-                    }
+                    adminClient.close();
                 }
             }
             throw e;