[HDDS-7204] EC: Schedule UnderReplicatedProcessor and OverReplicatedProcessor threads in RM instead of StorageContainerManager

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.3.0
• Component/s: SCM
• Labels: pull-request-available
• Target Version/s: 1.3.0
• Assignee: Siddhant Sangwan
• Reporter: Siddhant Sangwan
• Created: 07/Sep/22 06:22
• Updated: 05/May/23 15:22
• Resolved: 09/Sep/22 15:38

Description

Currently, UnderReplicatedProcessor and OverReplicatedProcessor are scheduled as BackgroundSCMService threads in StorageContainerManager. These processors directly depend on RM and should be managed by RM. This Jira moves their initialisation to the ReplicationManager class.

Under and Over Replicated processors don't check whether they should continue running when processing their queues. This is a potential bug - they won't get interrupted when looping through a very large number of containers. This is fixed by including a check for ReplicationManager#shouldRun() inside the loop.

Issue Links

• links to: GitHub Pull Request #3738