[ 
https://issues.apache.org/jira/browse/HBASE-29214?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Duo Zhang updated HBASE-29214:
------------------------------
    Component/s: Admin

> Typo in AsyncMasterRequestRpcRetryingCaller which makes us fail to clear the 
> master stub cache
> ----------------------------------------------------------------------------------------------
>
>                 Key: HBASE-29214
>                 URL: https://issues.apache.org/jira/browse/HBASE-29214
>             Project: HBase
>          Issue Type: Bug
>          Components: Admin, asyncclient, Client
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 2.7.0, 3.0.0-beta-2, 2.6.3, 2.5.12
>
>
> {code}
>   @Override
>   protected void doCall() {
>     addListener(conn.getMasterStub(), (stub, error) -> {
>       if (error != null) {
>         onError(error, () -> "Get async master stub failed", err -> {
>         });
>         return;
>       }
>       resetCallTimeout();
>       addListener(callable.call(controller, stub), (result, error2) -> {
>         if (error2 != null) {
>           onError(error2, () -> "Call to master failed",
>             err -> clearMasterStubCacheOnError(stub, error2)); // <==== this 
> line
>           return;
>         }
>         future.complete(result);
>       });
>     });
>   }
> {code}
> It should be err -> clearMasterStubCacheOnError(stub, err), so in 
> clearMasterStubCacheOnError we can get the unwrapped exception, and then we 
> can decide whether to clear the master cache correctly.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to