[GitHub] [accumulo] ddanielr commented on pull request #3234: update wait for tservers to allow infinite wait

["ddanielr (via GitHub)" <gi...@apache.org>]

ddanielr commented on PR #3234:
URL: https://github.com/apache/accumulo/pull/3234#issuecomment-1464486152

   Given that the intervals just need to be changed, wouldn't just changing the number of retries and the increment variable accomplish that without redoing the logic? 
   
   ```
       long retries = 10;
       long waitPeriod = maxWait / retries;
       long waitIncrement = 0;
       
       if (maxWait <= 0) {
         log.info("tserver availability check set to block indefinitely, To change, set {} > 0.",
             Property.MASTER_STARTUP_TSERVER_AVAIL_MAX_WAIT.getKey());
         maxWait = Long.MAX_VALUE;
         
         // Change retry logic for incremental backoff and logging due to indefinite blocking
         retries = maxWait;
         waitPeriod = maxWait;
         waitIncrement = 15_000;
       }
    ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org