slfan1989 commented on code in PR #6924:
URL: https://github.com/apache/ozone/pull/6924#discussion_r1676568864


##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/block/DeletedBlockLogImpl.java:
##########
@@ -426,6 +426,11 @@ public void onMessage(
 
     DatanodeDetails details = deleteBlockStatus.getDatanodeDetails();
     UUID dnId = details.getUuid();
+    UUID dnIdLocal = UUID.fromString(dnId.toString());
+    if (LOG.isDebugEnabled()) {
+      LOG.debug("remoteId = {}, localDnId = {}, remoteDnId == localDnId[{}]",
+          dnId, dnIdLocal, (dnId == dnIdLocal));

Review Comment:
   @adoroszlai Thank you very much for your reply! Your comment on object 
comparison is correct; I should use the equal method for comparison.
   
   > Is it possible that it can proceed simply because SCM was restarted?
   
   I still have some questions about this issue. Later, I will provide a 7-day 
Grafana monitoring report. The problem we're facing is that while SCM allows 
deletions, the deletion queue fails to decrease after a period of time. 
Restarting SCM temporarily restores normal deletion operations, but then the 
deletion queue stops decreasing again. However, since applying this PR, 
deletions have been ongoing for 4 days now. So, I might have mistakenly 
attributed this PR's effect, but there could be other issues at play.
   



-- 
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...@ozone.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@ozone.apache.org
For additional commands, e-mail: issues-h...@ozone.apache.org

Reply via email to