[GitHub] [hbase] sguggilam commented on pull request #1755: HBASE-24069 Provide an ExponentialBackOffPolicy sleep between failed …

2020-05-29 Thread GitBox


sguggilam commented on pull request #1755:
URL: https://github.com/apache/hbase/pull/1755#issuecomment-636253821


   @bharathv  pushed the latest changes incorporating review comments, please 
check



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.

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




[GitHub] [hbase] sguggilam commented on pull request #1755: HBASE-24069 Provide an ExponentialBackOffPolicy sleep between failed …

2020-05-29 Thread GitBox


sguggilam commented on pull request #1755:
URL: https://github.com/apache/hbase/pull/1755#issuecomment-636119423


   @apurtell @bharathv  Incorporated the code review comments, please take a 
look
   
   Also the Assignment Manager V2 already have the retry with back off logic 
for failed open/close requests 
(https://github.com/apache/hbase/blob/branch-2/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/TransitRegionStateProcedure.java#L360)
 , so the change is not needed for branch-2 and above. 



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.

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




[GitHub] [hbase] sguggilam commented on pull request #1755: HBASE-24069 Provide an ExponentialBackOffPolicy sleep between failed …

2020-05-21 Thread GitBox


sguggilam commented on pull request #1755:
URL: https://github.com/apache/hbase/pull/1755#issuecomment-632369851


   @apurtell  for review



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.

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