Repository: hbase
Updated Branches:
  refs/heads/branch-1.3 6b04084ab -> e4c8a858b


HBASE-17817 add table name to output (if available) when removing coprocessors

Amending-Author: Sean Busbey <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/e4c8a858
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/e4c8a858
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/e4c8a858

Branch: refs/heads/branch-1.3
Commit: e4c8a858b1957ab1123d0daf5a21507c7737f519
Parents: 6b04084
Author: Steen Manniche <[email protected]>
Authored: Tue Apr 11 17:48:50 2017 +0200
Committer: Sean Busbey <[email protected]>
Committed: Fri Apr 28 11:54:46 2017 -0500

----------------------------------------------------------------------
 .../apache/hadoop/hbase/coprocessor/CoprocessorHost.java | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/e4c8a858/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
----------------------------------------------------------------------
diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
index 30051d1..8271626 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorHost.java
@@ -559,8 +559,15 @@ public abstract class CoprocessorHost<E extends 
CoprocessorEnvironment> {
       // server is configured to abort.
       abortServer(env, e);
     } else {
-      LOG.error("Removing coprocessor '" + env.toString() + "' from " +
-          "environment because it threw:  " + e,e);
+      // If available, pull a table name out of the environment
+      if(env instanceof RegionCoprocessorEnvironment) {
+        String tableName = 
((RegionCoprocessorEnvironment)env).getRegionInfo().getTable().getNameAsString();
+        LOG.error("Removing coprocessor '" + env.toString() + "' from table 
'"+ tableName + "'", e);
+      } else {
+        LOG.error("Removing coprocessor '" + env.toString() + "' from " +
+                "environment",e);
+      }
+
       coprocessors.remove(env);
       try {
         shutdown(env);

Reply via email to