This is an automated email from the ASF dual-hosted git repository.
agupta pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new a605c284215 HDDS-13489. Fix SCMBlockdeleting unnecessary iteration in
corner case. (#8847)
a605c284215 is described below
commit a605c2842153bed35fa2ef867fd12206b848aab8
Author: Ashish Kumar <117710273+ashishkuma...@users.noreply.github.com>
AuthorDate: Wed Jul 23 02:46:27 2025 +0530
HDDS-13489. Fix SCMBlockdeleting unnecessary iteration in corner case.
(#8847)
---
.../java/org/apache/hadoop/hdds/scm/block/DeletedBlockLogImpl.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/block/DeletedBlockLogImpl.java
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/block/DeletedBlockLogImpl.java
index 48ffadfe76d..c94036b9cbc 100644
---
a/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/block/DeletedBlockLogImpl.java
+++
b/hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/block/DeletedBlockLogImpl.java
@@ -478,11 +478,11 @@ public DatanodeDeletedBlockTransactions getTransactions(
Set<ContainerReplica> replicas = containerManager
.getContainerReplicas(
ContainerID.valueOf(txn.getContainerID()));
- if (checkInadequateReplica(replicas, txn, dnList)) {
+ if (!checkInadequateReplica(replicas, txn, dnList)) {
+ getTransaction(txn, transactions, replicas, commandStatus,
maxDeleteBlocksPerDatanode);
+ } else {
metrics.incrSkippedTransaction();
- continue;
}
- getTransaction(txn, transactions, replicas, commandStatus,
maxDeleteBlocksPerDatanode);
} else if (txn.getCount() >= maxRetry ||
containerManager.getContainer(id).isOpen()) {
metrics.incrSkippedTransaction();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@ozone.apache.org
For additional commands, e-mail: commits-h...@ozone.apache.org
