ChenSammi commented on code in PR #6113:
URL: https://github.com/apache/ozone/pull/6113#discussion_r1548990350


##########
hadoop-ozone/integration-test/src/test/java/org/apache/hadoop/ozone/container/server/TestContainerServer.java:
##########
@@ -192,6 +195,9 @@ private static HddsDispatcher 
createDispatcher(DatanodeDetails dd, UUID scmId,
     conf.set(OZONE_METADATA_DIRS, TEST_DIR);
     VolumeSet volumeSet = new MutableVolumeSet(dd.getUuidString(), conf, null,
         StorageVolume.VolumeType.DATA_VOLUME, null);
+    Path tempDir = Files.createTempDirectory("");

Review Comment:
   This tempDir should be explicitly deleted, other wise it will leak disk 
space on CI server, resulting out of space error at last. 
   Same for change in TestSecureContainerServer.java.



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