ArafatKhan2198 commented on code in PR #10198:
URL: https://github.com/apache/ozone/pull/10198#discussion_r3215080951


##########
hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/ContainerEndpoint.java:
##########
@@ -447,6 +448,11 @@ private Response getUnhealthyContainersFromSchema(
     for (UnhealthyContainersSummary s : summary) {
       response.setSummaryCount(s.getContainerState(), s.getCount());
     }
+    
+    // Also include the quasi-closed count in the summary for the frontend 
Highlights tab
+    long quasiClosedCount = 
containerManager.getContainerStateCount(HddsProtos.LifeCycleState.QUASI_CLOSED);

Review Comment:
   Fixed. Removed `quasiClosedCount` from `UnhealthyContainersResponse` and the 
injection from `getUnhealthyContainersFromSchema`. On the frontend, the 
Highlights card now gets the quasi-closed count via an independent background 
call to `/api/v1/containers/quasiClosed?limit=1` on page load and on full 
refresh, keeping the unhealthy DB query path completely separate from the 
in-memory quasi-closed count.



-- 
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]

Reply via email to