ACCUMULO-3928 Use slf4j's fancy variable replacement
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/ba300baa Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/ba300baa Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/ba300baa Branch: refs/heads/master Commit: ba300baaa4826d21d784d727ba115388013d9fc8 Parents: 6243e6e Author: Josh Elser <els...@apache.org> Authored: Thu Jul 2 15:49:09 2015 -0400 Committer: Josh Elser <els...@apache.org> Committed: Thu Jul 2 15:49:09 2015 -0400 ---------------------------------------------------------------------- .../server/tabletserver/LargestFirstMemoryManager.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/accumulo/blob/ba300baa/server/base/src/main/java/org/apache/accumulo/server/tabletserver/LargestFirstMemoryManager.java ---------------------------------------------------------------------- diff --git a/server/base/src/main/java/org/apache/accumulo/server/tabletserver/LargestFirstMemoryManager.java b/server/base/src/main/java/org/apache/accumulo/server/tabletserver/LargestFirstMemoryManager.java index 865e775..d73c46e 100644 --- a/server/base/src/main/java/org/apache/accumulo/server/tabletserver/LargestFirstMemoryManager.java +++ b/server/base/src/main/java/org/apache/accumulo/server/tabletserver/LargestFirstMemoryManager.java @@ -174,7 +174,7 @@ public class LargestFirstMemoryManager implements MemoryManager { for (TabletState ts : tablets) { // Make sure that the table still exists if (!tableExists(instance, ts.getExtent().getTableId().toString())) { - log.info("Ignoring extent for deleted table: " + ts.getExtent()); + log.trace("Ignoring extent for deleted table: {}", ts.getExtent()); continue; } @@ -193,9 +193,7 @@ public class LargestFirstMemoryManager implements MemoryManager { } catch (IllegalArgumentException e) { Throwable cause = e.getCause(); if (null != cause && cause instanceof TableNotFoundException) { - if (log.isTraceEnabled()) { - log.trace("Ignoring extent for deleted table: " + ts.getExtent()); - } + log.trace("Ignoring extent for deleted table: {}", ts.getExtent()); // The table might have been deleted during the iteration of the tablets // We just want to eat this exception, do nothing with this tablet, and continue
