accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From els...@apache.org
Subject [7/8] accumulo git commit: ACCUMULO-3928 Use slf4j's fancy variable replacement
Date Thu, 02 Jul 2015 20:25:34 GMT
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/1.7
Commit: ba300baaa4826d21d784d727ba115388013d9fc8
Parents: 6243e6e
Author: Josh Elser <elserj@apache.org>
Authored: Thu Jul 2 15:49:09 2015 -0400
Committer: Josh Elser <elserj@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


Mime
View raw message