lhotari commented on a change in pull request #10032:
URL: https://github.com/apache/pulsar/pull/10032#discussion_r601061199
##########
File path:
pulsar-broker/src/test/java/org/apache/pulsar/io/PulsarFunctionE2ETest.java
##########
@@ -457,12 +458,6 @@ private void testE2EPulsarFunction(String jarFilePathUrl)
throws Exception {
// make sure subscriptions are cleanup
assertEquals(admin.topics().getStats(sourceTopic).subscriptions.size(), 0);
});
-
- // make sure all temp files are deleted
- File dir = new File(System.getProperty("java.io.tmpdir"));
- File[] foundFiles = dir.listFiles((dir1, name) ->
name.startsWith("function"));
-
- Assert.assertEquals(foundFiles.length, 0, "Temporary files left over:
" + Arrays.asList(foundFiles));
Review comment:
Thanks for the advice @jerrypeng . I'll revisit the changes and return
the checks.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
