devmadhuu commented on code in PR #4764:
URL: https://github.com/apache/ozone/pull/4764#discussion_r1231787811
##########
hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/OMDBInsightEndpoint.java:
##########
@@ -211,10 +254,53 @@ public Response getOpenKeyInfo(
break;
}
}
+ // Populate the keysSummary map
+ createKeysSummaryForOpenKey(keysSummary);
+
+ openKeyInsightInfo.setKeysSummary(keysSummary);
+
openKeyInsightInfo.setLastKey(lastKey);
Review Comment:
This seems incorrect, because lastKey will still be overridden from each
other (FSO vs NonFSO). We should provide only 1 argument in API end point for
type of buckets (FSO or NonFSO) at a time.
--
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]
