Repository: hbase Updated Branches: refs/heads/branch-1.3 d32230d0b -> 43626fc06
Revert "NPE in RpcServer causing intermittent UT failure of TestMasterReplication#testHFileCyclicReplication" This reverts commit d32230d0b5a4706b625cc7ac7ee7d28f44bd7b85. Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/39a4c566 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/39a4c566 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/39a4c566 Branch: refs/heads/branch-1.3 Commit: 39a4c56690eeeb2bb5ffaa0f3c8f6759b4fb3fb2 Parents: d32230d Author: Yu Li <[email protected]> Authored: Sat Jul 9 02:06:29 2016 +0800 Committer: Yu Li <[email protected]> Committed: Sat Jul 9 02:06:29 2016 +0800 ---------------------------------------------------------------------- .../src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/39a4c566/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java index 404f2ec..02fe1df 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java @@ -2298,13 +2298,7 @@ public class RpcServer implements RpcServerInterface, ConfigurationObserver { // The above callBlockingMethod will always return a SE. Strip the SE wrapper before // putting it on the wire. Its needed to adhere to the pb Service Interface but we don't // need to pass it over the wire. - if (e instanceof ServiceException) { - if (e.getCause() == null) { - LOG.debug("Caught a ServiceException with null cause", e); - } else { - e = e.getCause(); - } - } + if (e instanceof ServiceException) e = e.getCause(); // increment the number of requests that were exceptions. metrics.exception(e);
