xiaoyuyao commented on a change in pull request #2001:
URL: https://github.com/apache/ozone/pull/2001#discussion_r589784305
##########
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:
Can we add a new listContainer API with state and keep the existing one
for better backward compatibility?
----------------------------------------------------------------
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]
