S3 aux object deletion fixes #3921

IQSS · Aug 15, 2017 · 585b2b5 · 585b2b5
1 parent 3c0f8dd
commit 585b2b5
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/src/main/java/edu/harvard/iq/dataverse/dataaccess/S3AccessIO.java b/src/main/java/edu/harvard/iq/dataverse/dataaccess/S3AccessIO.java
@@ -467,21 +467,18 @@ public void deleteAllAuxObjects() throws IOException {
         }
         //Check if the list of auxiliary files for a data file is empty
         if (keys.isEmpty()) {
+            logger.fine("S3AccessIO: No auxiliary objects to delete.");
             return;
         }
         multiObjectDeleteRequest.setKeys(keys);
 
-        if (keys.isEmpty()){
-            logger.warning("S3AccessIO: no auxiliary objects to delete..");
-        } else {
-            logger.info("Trying to delete auxiliary files...");
+        logger.info("Trying to delete auxiliary files...");
             try {
                 s3.deleteObjects(multiObjectDeleteRequest);
             } catch (MultiObjectDeleteException e) {
                 logger.warning("S3AccessIO: Unable to delete auxilary objects" + e.getMessage());
                 throw new IOException("S3AccessIO: Failed to delete one or more auxiliary objects.");
             }
-        }
     }
 
     //TODO: Do we need this?