hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Yu Li (JIRA)" <j...@apache.org>
Subject [jira] [Created] (HBASE-14521) Uniform the semantic of hbase.client.retries.number
Date Wed, 30 Sep 2015 07:37:04 GMT
Yu Li created HBASE-14521:
-----------------------------

             Summary: Uniform the semantic of hbase.client.retries.number
                 Key: HBASE-14521
                 URL: https://issues.apache.org/jira/browse/HBASE-14521
             Project: HBase
          Issue Type: Bug
    Affects Versions: 1.1.2, 0.98.14
            Reporter: Yu Li
            Assignee: Yu Li


>From name of the _hbase.client.retries.number_ property, it should be the number of maximum
*retries*, or say if we set the property to 1, there should be 2 attempts in total. However,
there're two different semantics when using it in current code base.

For example, in ConnectionImplementation#locateRegionInMeta:
{code}
    int localNumRetries = (retry ? numTries : 1);

    for (int tries = 0; true; tries++) {
      if (tries >= localNumRetries) {
        throw new NoServerForRegionException("Unable to find region for "
            + Bytes.toStringBinary(row) + " in " + tableName +
            " after " + numTries + " tries.");
      }
{code}
the retries number is regarded as max times for *tries*

While in RpcRetryingCallerImpl#callWithRetries:
{code}
    for (int tries = 0;; tries++) {
      long expectedSleep;
      try {
        callable.prepare(tries != 0); // if called with false, check table status on ZK
        interceptor.intercept(context.prepare(callable, tries));
        return callable.call(getRemainingTime(callTimeout));
      } catch (PreemptiveFastFailException e) {
        throw e;
      } catch (Throwable t) {
        ...
        if (tries >= retries - 1) {
          throw new RetriesExhaustedException(tries, exceptions);
        }
{code}
it's regarded as exactly for *REtry* (try a call first with no condition and then check whether
to retry or exceeds maximum retry number)

This inconsistency will cause misunderstanding in usage, such as one of our customer set the
property to zero expecting one single call but finally received NoServerForRegionException.

We should uniform the semantic of the property, and I suggest to keep the original one for
retry rather than total tries.



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

Mime
View raw message