[hbase] branch branch-2 updated: HBASE-24524 SyncTable logging improvements (#1878)

[wchevreuil]

This is an automated email from the ASF dual-hosted git repository.

wchevreuil pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new ef984cb  HBASE-24524 SyncTable logging improvements (#1878)
ef984cb is described below

commit ef984cb18eb21f195b8b32a3fccd61cdaf4ac037
Author: Wellington Ramos Chevreuil <wchevre...@apache.org>
AuthorDate: Wed Jun 10 09:36:30 2020 +0100

    HBASE-24524 SyncTable logging improvements (#1878)
    
    Signed-off-by: Jan Hentschel <jan.hentsc...@ultratendency.com>
    Signed-off-by: Viraj Jasani <vjas...@apache.org>
    (cherry picked from commit d8247ebae347915c3fbc73f8ab6c39b0984826b0)
---
 .../java/org/apache/hadoop/hbase/mapreduce/SyncTable.java    | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git 
a/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/SyncTable.java
 
b/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/SyncTable.java
index cf9241d..6722501 100644
--- 
a/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/SyncTable.java
+++ 
b/hbase-mapreduce/src/main/java/org/apache/hadoop/hbase/mapreduce/SyncTable.java
@@ -363,16 +363,16 @@ public class SyncTable extends Configured implements Tool 
{
         boolean rowMatched;
         int rowComparison = compareRowKeys(nextSourceRow, nextTargetRow);
         if (rowComparison < 0) {
-          if (LOG.isInfoEnabled()) {
-            LOG.info("Target missing row: " + Bytes.toHex(nextSourceRow));
+          if (LOG.isDebugEnabled()) {
+            LOG.debug("Target missing row: " + Bytes.toString(nextSourceRow));
           }
           context.getCounter(Counter.TARGETMISSINGROWS).increment(1);
 
           rowMatched = syncRowCells(context, nextSourceRow, sourceCells, 
EMPTY_CELL_SCANNER);
           nextSourceRow = sourceCells.nextRow();  // advance only source to 
next row
         } else if (rowComparison > 0) {
-          if (LOG.isInfoEnabled()) {
-            LOG.info("Source missing row: " + Bytes.toHex(nextTargetRow));
+          if (LOG.isDebugEnabled()) {
+            LOG.debug("Source missing row: " + Bytes.toString(nextTargetRow));
           }
           context.getCounter(Counter.SOURCEMISSINGROWS).increment(1);
 
@@ -557,10 +557,10 @@ public class SyncTable extends Configured implements Tool 
{
             if (LOG.isDebugEnabled()) {
               LOG.debug("Different values: ");
               LOG.debug("  source cell: " + sourceCell
-                  + " value: " + Bytes.toHex(sourceCell.getValueArray(),
+                  + " value: " + Bytes.toString(sourceCell.getValueArray(),
                       sourceCell.getValueOffset(), 
sourceCell.getValueLength()));
               LOG.debug("  target cell: " + targetCell
-                  + " value: " + Bytes.toHex(targetCell.getValueArray(),
+                  + " value: " + Bytes.toString(targetCell.getValueArray(),
                       targetCell.getValueOffset(), 
targetCell.getValueLength()));
             }
             context.getCounter(Counter.DIFFERENTCELLVALUES).increment(1);