avijayanhwx commented on a change in pull request #648: HDDS-1340. Add List
Containers API for Recon
URL: https://github.com/apache/hadoop/pull/648#discussion_r269382496
##########
File path:
hadoop-ozone/ozone-recon/src/test/java/org/apache/hadoop/ozone/recon/api/TestContainerKeyService.java
##########
@@ -198,6 +198,38 @@ public void testGetKeysForContainer() throws Exception {
assertTrue(keyMetadataList.isEmpty());
}
+ @Test
+ public void testGetContainerIDList() throws Exception {
+ //Take snapshot of OM DB and copy over to Recon OM DB.
+ DBCheckpoint checkpoint = omMetadataManager.getStore()
Review comment:
Why are we taking DB snapshot if we are not writing anything new to OM DB?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]