[ https://issues.apache.org/jira/browse/HDDS-1690?focusedWorklogId=263089&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-263089 ]
ASF GitHub Bot logged work on HDDS-1690: ---------------------------------------- Author: ASF GitHub Bot Created on: 19/Jun/19 14:38 Start Date: 19/Jun/19 14:38 Worklog Time Spent: 10m Work Description: nandakumar131 commented on pull request #986: [HDDS-1690] ContainerController should provide a way to retrieve cont… URL: https://github.com/apache/hadoop/pull/986#discussion_r295333582 ########## File path: hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/ozoneimpl/ContainerController.java ########## @@ -140,4 +141,8 @@ private Handler getHandler(final Container container) { public Iterator<Container> getContainers() { return containerSet.getContainerIterator(); } + + public Iterator<Container> getContainersForVolume(String volumeUuid) { Review comment: Instead of `String` can we use `HddsVolume` as method argument? Then we can also rename the method to `public Iterator<Container> getContainers(HddsVolume volume)` ---------------------------------------------------------------- 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: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 263089) Time Spent: 1h (was: 50m) > ContainerController should provide a way to retrieve containers per volume > -------------------------------------------------------------------------- > > Key: HDDS-1690 > URL: https://issues.apache.org/jira/browse/HDDS-1690 > Project: Hadoop Distributed Data Store > Issue Type: Sub-task > Components: Ozone Datanode > Affects Versions: 0.4.0 > Reporter: Hrishikesh Gadre > Assignee: Hrishikesh Gadre > Priority: Major > Labels: pull-request-available > Time Spent: 1h > Remaining Estimate: 0h > > Ozone data scrubber needs to start a dedicated thread per volume to perform > checksum validation for all the containers in the given volume. But currently > ContainerController does not expose an API to fetch the list of containers > for a given volume. This Jira is to add the relevant API to > ContainerController class so that we can retrieve list of containers for a > given HDDS volume. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org