Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-19148 Reevaluate default values of configurations
  3. HBASE-19653

Reduce the default hbase.client.start.log.errors.counter

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Trivial
    • Resolution: Fixed
    • None
    • 2.0.0-beta-1, 2.0.0
    • Operability
    • None
    • Reviewed

    Description

      As we reduce the default retries number to 10 and now the default start log errors counter is 9. So it only log the error at the last retry. So we should reduce the default hbase.client.start.log.errors.counter, too.

        /**
         * Configure the number of failures after which the client will start logging. A few failures
         * is fine: region moved, then is not opened, then is overloaded. We try to have an acceptable
         * heuristic for the number of errors we don't log. 9 was chosen because we wait for 1s at
         * this stage.
         */
        public static final String START_LOG_ERRORS_AFTER_COUNT_KEY =
            "hbase.client.start.log.errors.counter";
        public static final int DEFAULT_START_LOG_ERRORS_AFTER_COUNT = 9;
      
      public static final int [] RETRY_BACKOFF = {1, 2, 3, 5, 10, 20, 40, 100, 100, 100, 100, 200, 200};
      public static final long DEFAULT_HBASE_CLIENT_PAUSE = 100;
      

      The default pause is 100ms and 100ms * 10 = 1s. The old comment of DEFAULT_START_LOG_ERRORS_AFTER_COUNT seems not right...

      Open this issue to reduce the default hbase.client.start.log.errors.counter to 5.

      Attachments

        1. HBASE-19653.master.001.patch
          1 kB
          Guanghao Zhang

        Activity

          People

            zghao Guanghao Zhang
            zghao Guanghao Zhang
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: