ChenSammi commented on a change in pull request #2001:
URL: https://github.com/apache/ozone/pull/2001#discussion_r589991592
##########
File path:
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/scm/client/ScmClient.java
##########
@@ -98,12 +98,13 @@ void deleteContainer(long containerId, Pipeline pipeline,
boolean force)
*
* @param startContainerID start containerID.
* @param count count must be {@literal >} 0.
+ * @param state Container of this state will be returned.
*
* @return a list of pipeline.
* @throws IOException
*/
List<ContainerInfo> listContainer(long startContainerID,
Review comment:
Sure. I will add a new listContainer API. Actually that's the way of
my first patch. Then I thought a consolidated API looks like more neat.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
