This is an automated email from the ASF dual-hosted git repository.
zhangduo pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2 by this push:
new 49af9d8 HBASE-25073 Should not use
XXXService.Interface.class.getSimpleName as stub key prefix in
AsyncConnectionImpl (#2443)
49af9d8 is described below
commit 49af9d8e91c4771c8abee9537f5b5bb31b57ff39
Author: 王伟 <[email protected]>
AuthorDate: Wed Sep 23 14:36:00 2020 +0800
HBASE-25073 Should not use XXXService.Interface.class.getSimpleName as stub
key prefix in AsyncConnectionImpl (#2443)
Signed-off-by: Duo Zhang <[email protected]>
---
.../main/java/org/apache/hadoop/hbase/client/AsyncConnectionImpl.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionImpl.java
b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionImpl.java
index c067c1c..f76e084 100644
---
a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionImpl.java
+++
b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncConnectionImpl.java
@@ -243,7 +243,7 @@ class AsyncConnectionImpl implements AsyncConnection {
ClientService.Interface getRegionServerStub(ServerName serverName) throws
IOException {
return ConcurrentMapUtils.computeIfAbsentEx(rsStubs,
- getStubKey(ClientService.Interface.class.getSimpleName(), serverName,
hostnameCanChange),
+ getStubKey(ClientService.getDescriptor().getName(), serverName,
hostnameCanChange),
() -> createRegionServerStub(serverName));
}
@@ -257,7 +257,7 @@ class AsyncConnectionImpl implements AsyncConnection {
AdminService.Interface getAdminStub(ServerName serverName) throws
IOException {
return ConcurrentMapUtils.computeIfAbsentEx(adminSubs,
- getStubKey(AdminService.Interface.class.getSimpleName(), serverName,
hostnameCanChange),
+ getStubKey(AdminService.getDescriptor().getName(), serverName,
hostnameCanChange),
() -> createAdminServerStub(serverName));
}