You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by wc...@apache.org on 2020/06/10 08:36:43 UTC
[hbase] branch master updated: HBASE-24524 SyncTable logging
improvements (#1878)
This is an automated email from the ASF dual-hosted git repository.
wchevreuil pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/master by this push:
new d8247eb HBASE-24524 SyncTable logging improvements (#1878)
d8247eb is described below
commit d8247ebae347915c3fbc73f8ab6c39b0984826b0
Author: Wellington Ramos Chevreuil <wc...@apache.org>
AuthorDate: Wed Jun 10 09:36:30 2020 +0100
HBASE-24524 SyncTable logging improvements (#1878)
Signed-off-by: Jan Hentschel <ja...@ultratendency.com>
Signed-off-by: Viraj Jasani <vj...@apache.org>
---
.../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 bb20061..b60f828 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
@@ -364,16 +364,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);
@@ -558,10 +558,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);