hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HBASE-16149) Log the underlying RPC exception in RpcRetryingCallerImpl
Date Thu, 30 Jun 2016 13:00:14 GMT

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

stack updated HBASE-16149:
--------------------------
       Resolution: Fixed
     Hadoop Flags: Reviewed
    Fix Version/s: 2.0.0
           Status: Resolved  (was: Patch Available)

Pushed to master branch. Doesn't apply cleanly to branch-1. If you put up a patch for branch-1,
I'll commit it. Thanks [~jerryhe] Good one.

> Log the underlying RPC exception in RpcRetryingCallerImpl 
> ----------------------------------------------------------
>
>                 Key: HBASE-16149
>                 URL: https://issues.apache.org/jira/browse/HBASE-16149
>             Project: HBase
>          Issue Type: Improvement
>    Affects Versions: 1.2.0
>            Reporter: Jerry He
>            Assignee: Jerry He
>            Priority: Minor
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16149.patch
>
>
> In RpcRetryingCallerImpl:
> {code}
>   public T callWithRetries(RetryingCallable<T> callable, int callTimeout)
>   throws IOException, RuntimeException {
> ...
>     for (int tries = 0;; tries++) {
>       try {
> ...
>         return callable.call(getTimeout(callTimeout));
> ...
>       } catch (Throwable t) {
>         ExceptionUtil.rethrowIfInterrupt(t);
>         if (tries > startLogErrorsCnt) {
>           LOG.info("Call exception, tries=" + tries + ", maxAttempts=" + maxAttempts
+ ", started="
>               + (EnvironmentEdgeManager.currentTime() - tracker.getStartTime()) + " ms
ago, "
>               + "cancelled=" + cancelled.get() + ", msg="
>               + callable.getExceptionMessageAdditionalDetail());
>         }
> ...
> {code}
> We log the callable.getExceptionMessageAdditionalDetail() msg. But callable.getExceptionMessageAdditionalDetail()
may not provide the underlying cause..
> For example, in AbstractRegionServerCallable, 
> {code}
>   public String getExceptionMessageAdditionalDetail() {
>     return "row '" + Bytes.toString(row) + "' on table '" + tableName + "' at " + location;
>   }
> {code}
> Let's add the underlying exception cause to the message as well.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message