adoroszlai commented on code in PR #5520:
URL: https://github.com/apache/ozone/pull/5520#discussion_r1377191157


##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/interfaces/BlockManager.java:
##########
@@ -34,7 +34,7 @@ public interface BlockManager {
    * Puts or overwrites a block.
    *
    * @param container - Container for which block need to be added.
-   * @param data     - Block Data.
+   * @param data - Block Data.
    * @return length of the Block.
    * @throws IOException

Review Comment:
   Please also remove such `@throws` tags that only mention the exception type, 
but not when or why it would be thrown.  The same information is already known 
from the methods' `throws` declaration.



-- 
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]

Reply via email to