Amend HBASE-14771 RpcServer#getRemoteAddress always returns null This change has been reported to cause Phoenix's PhoenixIndexRpcSchedulerTest to fail with a NPE
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/79588240 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/79588240 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/79588240 Branch: refs/heads/master Commit: 79588240c60bf422fb9d9a74e0edd8b47a66b9f9 Parents: 2d2fdd5 Author: Andrew Purtell <[email protected]> Authored: Fri Jan 15 17:32:43 2016 -0800 Committer: Andrew Purtell <[email protected]> Committed: Fri Jan 15 19:27:18 2016 -0800 ---------------------------------------------------------------------- .../src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/79588240/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 cdc97bf..bd3342b 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 @@ -2603,7 +2603,7 @@ public class RpcServer implements RpcServerInterface, ConfigurationObserver { */ public static InetAddress getRemoteIp() { Call call = CurCall.get(); - if (call != null && call.connection.socket != null) { + if (call != null && call.connection != null && call.connection.socket != null) { return call.connection.socket.getInetAddress(); } return null;
