You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by el...@apache.org on 2015/07/02 22:25:33 UTC
[6/8] accumulo git commit: ACCUMULO-3928 Use slf4j's fancy variable
replacement
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 <el...@apache.org>
Authored: Thu Jul 2 15:49:09 2015 -0400
Committer: Josh Elser <el...@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