You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by la...@apache.org on 2014/02/05 22:52:52 UTC

svn commit: r1564950 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java

Author: larsh
Date: Wed Feb  5 21:52:52 2014
New Revision: 1564950

URL: http://svn.apache.org/r1564950
Log:
HBASE-10470 Import generates huge log file while importing large amounts of data. (Vasu Mariyala)

Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java?rev=1564950&r1=1564949&r2=1564950&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/mapreduce/Import.java Wed Feb  5 21:52:52 2014
@@ -247,13 +247,12 @@ public class Import {
     // apply the filter and skip this kv if the filter doesn't apply
     if (filter != null) {
       Filter.ReturnCode code = filter.filterKeyValue(kv);
-      System.out.println("Filter returned:" + code);
+      if (LOG.isTraceEnabled()) {
+        LOG.trace("Filter returned:" + code + " for the key value:" + kv);
+      }
       // if its not an accept type, then skip this kv
       if (!(code.equals(Filter.ReturnCode.INCLUDE) || code
           .equals(Filter.ReturnCode.INCLUDE_AND_NEXT_COL))) {
-        if (LOG.isDebugEnabled()) {
-          System.out.println("Skipping key: " + kv + " from filter decision: " + code);
-        }
         return null;
       }
     }