You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2013/03/23 22:05:29 UTC

svn commit: r1460231 - /hbase/branches/0.95/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java

Author: stack
Date: Sat Mar 23 21:05:28 2013
New Revision: 1460231

URL: http://svn.apache.org/r1460231
Log:
HBASE-8178 IPCUtil produces a LOT of WARN log spew

Modified:
    hbase/branches/0.95/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java

Modified: hbase/branches/0.95/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java?rev=1460231&r1=1460230&r2=1460231&view=diff
==============================================================================
--- hbase/branches/0.95/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java (original)
+++ hbase/branches/0.95/hbase-client/src/main/java/org/apache/hadoop/hbase/ipc/IPCUtil.java Sat Mar 23 21:05:28 2013
@@ -102,9 +102,11 @@ class IPCUtil {
       os.close();
       if (poolCompressor != null) CodecPool.returnCompressor(poolCompressor);
     }
-    if (this.cellBlockBuildingInitialBufferSize < baos.size()) {
-      LOG.warn("Buffer grew from " + this.cellBlockBuildingInitialBufferSize +
-      " to " + baos.size());
+    if (LOG.isTraceEnabled()) {}
+      if (this.cellBlockBuildingInitialBufferSize < baos.size()) {
+        LOG.trace("Buffer grew from " + this.cellBlockBuildingInitialBufferSize +
+        " to " + baos.size());
+      }
     }
     return baos.getByteBuffer();
   }
@@ -243,4 +245,4 @@ class IPCUtil {
     Preconditions.checkArgument(totalSize < Integer.MAX_VALUE);
     return totalSize;
   }
-}
\ No newline at end of file
+}