siddhantsangwan commented on code in PR #4415:
URL: https://github.com/apache/ozone/pull/4415#discussion_r1141670989
##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/balancer/MoveManager.java:
##########
@@ -109,8 +109,11 @@ public enum MoveResult {
// TODO - Should pending ops notify under lock to allow MM to schedule a
// delete after the move, but before anything else can, eg RM?
- // TODO - these need to be config defined somewhere, probably in the balancer
- private static final long MOVE_DEADLINE = 1000 * 60 * 60; // 1 hour
+ /*
+ moveDeadline and replicationDeadline are set by ContainerBalancer.
+ */
+ private long moveDeadline = 1000 * 90 * 60;
+ private long replicationDeadline = 1000 * 60 * 60;
Review Comment:
Oh yeah, thanks for pointing this out. Renaming these.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
