From a969f8d7903d07ca1f6672db4b2a739c227e70a0 Mon Sep 17 00:00:00 2001 From: Michael Stack Date: Mon, 31 Oct 2016 14:37:51 -0700 Subject: [PATCH] HBASE-16969 RegionCoprocessorServiceExec should override the toString() for debugging (ChiaPing Tsai) --- .../hbase/client/RegionCoprocessorServiceExec.java | 14 ++++++++++++++ .../hadoop/hbase/CoordinatedStateManager.java | 2 +- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionCoprocessorServiceExec.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionCoprocessorServiceExec.java index 2d62332c22f..cd4716e9050 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionCoprocessorServiceExec.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegionCoprocessorServiceExec.java @@ -101,4 +101,18 @@ public class RegionCoprocessorServiceExec implements Row { Row other = (Row) obj; return compareTo(other) == 0; } + + @Override + public String toString() { + StringBuilder builder = new StringBuilder(); + builder.append("region:") + .append(Bytes.toStringBinary(region)) + .append(", startKey:") + .append(Bytes.toStringBinary(startKey)) + .append(", method:") + .append(method.getFullName()) + .append(", request:") + .append(request); + return builder.toString(); + } } diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/CoordinatedStateManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/CoordinatedStateManager.java index b7bfa751c24..bdb202d362c 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/CoordinatedStateManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/CoordinatedStateManager.java @@ -20,7 +20,7 @@ package org.apache.hadoop.hbase; import org.apache.hadoop.hbase.classification.InterfaceAudience; /** - * Implementations of this interface will keep and return to clients + * Implementations of this interface will keep and return to clients * implementations of classes providing API to execute * coordinated operations. This interface is client-side, so it does NOT * include methods to retrieve the particular interface implementations.