bbeaudreault commented on code in PR #5534: URL: https://github.com/apache/hbase/pull/5534#discussion_r1407920432
########## hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ReopenTableRegionsProcedure.java: ########## @@ -61,20 +69,36 @@ public class ReopenTableRegionsProcedure private List<HRegionLocation> regions = Collections.emptyList(); + private List<HRegionLocation> currentRegionBatch = Collections.emptyList(); + private RetryCounter retryCounter; + private final long reopenBatchBackoffMillis; Review Comment: I don't think so because they're just configuration. Probably preferable for the configuration to update on restart rather than be stuck on what it started with. Maybe see if other procedures take configurations like this -- 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: issues-unsubscr...@hbase.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org