You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by zy...@apache.org on 2018/07/10 04:38:39 UTC
[17/50] [abbrv] hbase git commit: HBASE-20810 Include the procedure
id in the exception message in HBaseAdmin for better debugging
HBASE-20810 Include the procedure id in the exception message in HBaseAdmin for better debugging
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/63477d62
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/63477d62
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/63477d62
Branch: refs/heads/HBASE-18477
Commit: 63477d6251a597065af4f9619ec68699721123a4
Parents: fe75f90
Author: zhangduo <zh...@apache.org>
Authored: Thu Jun 28 22:00:11 2018 +0800
Committer: zhangduo <zh...@apache.org>
Committed: Fri Jun 29 11:40:44 2018 +0800
----------------------------------------------------------------------
.../org/apache/hadoop/hbase/client/HBaseAdmin.java | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/63477d62/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
----------------------------------------------------------------------
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
index ac8d972..c7456dd 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
@@ -3363,7 +3363,7 @@ public class HBaseAdmin implements Admin {
private V result = null;
private final HBaseAdmin admin;
- private final Long procId;
+ protected final Long procId;
public ProcedureFuture(final HBaseAdmin admin, final Long procId) {
this.admin = admin;
@@ -3645,22 +3645,20 @@ public class HBaseAdmin implements Admin {
* @return a description of the operation
*/
protected String getDescription() {
- return "Operation: " + getOperationType() + ", "
- + "Table Name: " + tableName.getNameWithNamespaceInclAsString();
-
+ return "Operation: " + getOperationType() + ", " + "Table Name: " +
+ tableName.getNameWithNamespaceInclAsString() + ", procId: " + procId;
}
protected abstract class TableWaitForStateCallable implements WaitForStateCallable {
@Override
public void throwInterruptedException() throws InterruptedIOException {
- throw new InterruptedIOException("Interrupted while waiting for operation: "
- + getOperationType() + " on table: " + tableName.getNameWithNamespaceInclAsString());
+ throw new InterruptedIOException("Interrupted while waiting for " + getDescription());
}
@Override
public void throwTimeoutException(long elapsedTime) throws TimeoutException {
- throw new TimeoutException("The operation: " + getOperationType() + " on table: " +
- tableName.getNameAsString() + " has not completed after " + elapsedTime + "ms");
+ throw new TimeoutException(
+ getDescription() + " has not completed after " + elapsedTime + "ms");
}
}