Fix LOG.warn without context (#6924)

Alluxio · Mar 7, 2018 · 049041a · 049041a
1 parent b3f7e2c
commit 049041a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/core/server/master/src/main/java/alluxio/master/file/DefaultFileSystemMaster.java b/core/server/master/src/main/java/alluxio/master/file/DefaultFileSystemMaster.java
@@ -1504,7 +1504,7 @@ private List<Inode<?>> deleteInternal(LockedInodePath inodePath, boolean replaye
               failedToDelete = !ufsDeleter.delete(alluxioUriToDel, delInode);
             }
           } catch (InvalidPathException e) {
-            LOG.warn(e.getMessage());
+            LOG.warn("Failed to delete path from UFS: {}", e.getMessage());
           }
         }
         if (!failedToDelete) {

diff --git a/core/server/worker/src/main/java/alluxio/worker/block/SpaceReserver.java b/core/server/worker/src/main/java/alluxio/worker/block/SpaceReserver.java
@@ -102,16 +102,17 @@ private void reserveSpace() {
             mBlockWorker.freeSpace(Sessions.MIGRATE_DATA_SESSION_ID, reservedSpace, tierAlias);
           } catch (WorkerOutOfSpaceException | BlockDoesNotExistException
               | BlockAlreadyExistsException | InvalidWorkerStateException | IOException e) {
-            LOG.warn("SpaceReserver failed to free tier {} to {} bytes used", tierAlias,
-                reservedSpace, e.getMessage());
+            LOG.warn("SpaceReserver failed to free tier {} to {} bytes used for high watermarks: "
+                + "{}", tierAlias, reservedSpace, e.getMessage());
           }
         }
       } else {
         try {
           mBlockWorker.freeSpace(Sessions.MIGRATE_DATA_SESSION_ID, reservedSpace, tierAlias);
         } catch (WorkerOutOfSpaceException | BlockDoesNotExistException
             | BlockAlreadyExistsException | InvalidWorkerStateException | IOException e) {
-          LOG.warn(e.getMessage());
+          LOG.warn("SpaceReserver failed to free tier {} to {} bytes used: {}", tierAlias,
+              reservedSpace, e.getMessage());
         }
       }
     }