Skip to content

HDDS-14702. Make lazy source container replica deletion interval configurable#9837

Merged
ChenSammi merged 4 commits intoapache:masterfrom
Gargi-jais11:HDDS-14702
Mar 6, 2026
Merged

HDDS-14702. Make lazy source container replica deletion interval configurable#9837
ChenSammi merged 4 commits intoapache:masterfrom
Gargi-jais11:HDDS-14702

Conversation

@Gargi-jais11
Copy link
Contributor

What changes were proposed in this pull request?

Currently source container replica deletion is postponed by 1 hour after the container is successfully moved from source volume to destination volume during disk balancing.
This PR aims to make this configurable, and reduce the default value from 1 hour to 5 minutes.

<property>
    <name>hdds.datanode.disk.balancer.replica.deletion.delay</name>
    <value>5m</value>
    <tag>OZONE, DATANODE, DISKBALANCER</tag>
    <description>The delay after a container is successfully moved from source
      volume to destination volume before the source container replica is deleted.
      By default this is set to 5 minutes. Unit: ns, ms, s, m, h, d.
    </description>
  </property>

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-14702

How was this patch tested?

Passed existing test cases for replica deletion delay.

@Gargi-jais11 Gargi-jais11 marked this pull request as ready for review February 26, 2026 15:48
@Gargi-jais11
Copy link
Contributor Author

@ChenSammi Please review the patch.

Copy link
Contributor

@sreejasahithi sreejasahithi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Gargi-jais11 for working on this.

Gargi-jais11 and others added 2 commits March 2, 2026 10:48
…ozone/container/diskbalancer/DiskBalancerConfiguration.java

Co-authored-by: sreejasahithi <115860222+sreejasahithi@users.noreply.github.com>
Co-authored-by: sreejasahithi <115860222+sreejasahithi@users.noreply.github.com>
Copy link
Contributor

@sreejasahithi sreejasahithi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Gargi-jais11 Gargi-jais11 requested a review from ChenSammi March 2, 2026 10:44
Copy link
Contributor

@priyeshkaratha priyeshkaratha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes LGTM

Copy link
Contributor

@ChenSammi ChenSammi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Gargi-jais11 , and @sreejasahithi @priyeshkaratha for the review.

@ChenSammi ChenSammi merged commit 515869d into apache:master Mar 6, 2026
46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants