Copilot commented on code in PR #10704:
URL: https://github.com/apache/ozone/pull/10704#discussion_r3562195245
##########
hadoop-hdds/server-scm/src/test/java/org/apache/hadoop/hdds/scm/node/TestSCMNodeManager.java:
##########
@@ -218,21 +220,23 @@ SCMNodeManager createNodeManager(OzoneConfiguration
config)
*/
@Test
public void testScmHeartbeat()
- throws IOException, InterruptedException, AuthenticationException {
+ throws IOException, AuthenticationException {
Review Comment:
The Javadoc still lists `@throws` InterruptedException and `@throws`
TimeoutException, but testScmHeartbeat no longer declares (or triggers) either
exception. This is now inaccurate and may confuse readers/IDE inspections.
##########
hadoop-hdds/server-scm/src/test/java/org/apache/hadoop/hdds/scm/node/TestContainerPlacement.java:
##########
@@ -179,7 +181,7 @@ ContainerManager createContainerManager()
*/
@Test
public void testContainerPlacementCapacity() throws IOException,
- InterruptedException {
+ InterruptedException, TimeoutException {
Review Comment:
The test now throws TimeoutException via GenericTestUtils.waitFor, but the
Javadoc `@throws` list was not updated to mention it.
--
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]
