adoroszlai commented on PR #8348:
URL: https://github.com/apache/ozone/pull/8348#issuecomment-2835118437

   Thanks @chiacyu for the patch.
   
   Please add the property in `pom.xml` instead of duplicated 
`junit-platform.properties` files.
   
   
https://github.com/apache/ozone/blob/6f5e02a72b460d845ba243f040684f63c5d6eb8f/pom.xml#L1917-L1918
   
   Also, there are some leftover `@Timeout` annotations.  I think all of these 
can be removed.
   
   ```
   
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/TestGrpcOzoneManagerServer.java
   32:@Timeout(30)
   
   
hadoop-hdds/container-service/src/test/java/org/apache/hadoop/ozone/container/common/volume/TestVolumeSetDiskChecks.java
   76:@Timeout(30)
   
   
hadoop-hdds/container-service/src/test/java/org/apache/hadoop/ozone/container/common/volume/TestStorageVolumeChecker.java
   73:@Timeout(300)
   
   
hadoop-hdds/container-service/src/test/java/org/apache/hadoop/ozone/container/keyvalue/TestKeyValueContainerMarkUnhealthy.java
   53:@Timeout(600)
   
   
hadoop-hdds/server-scm/src/test/java/org/apache/hadoop/hdds/scm/TestHddsServerUtils.java
   50:@Timeout(300)
   
   
hadoop-ozone/s3gateway/src/test/java/org/apache/hadoop/ozone/protocolPB/TestGrpcOmTransport.java
   43:@Timeout(3000)
   
   
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/fsck/TestContainerMapper.java
   59:@Timeout(value = 300, unit = TimeUnit.SECONDS)
   
   
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOmInit.java
   34:@Timeout(300)
   
   
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/om/TestOMRatisSnapshots.java
   105:@Timeout(5000)
   
   
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/shell/TestOzoneTenantShell.java
   75:@Timeout(300)
   ```


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