ChenSammi commented on code in PR #8663:
URL: https://github.com/apache/ozone/pull/8663#discussion_r2157952509
##########
hadoop-hdds/server-scm/src/main/java/org/apache/hadoop/hdds/scm/container/ContainerManager.java:
##########
@@ -191,7 +191,8 @@ default ContainerInfo getMatchingContainer(long size,
String owner,
* @param owner - the user which requires space in its owned container
* @param pipeline - pipeline to which the container should belong.
* @param excludedContainerIDS - containerIds to be excluded.
- * @return ContainerInfo for the matching container.
+ * @return ContainerInfo for the matching container, or null if a container
could not be found and could not be
+ * allocated
Review Comment:
Use @Nullable, which is very straightforward.
--
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]
