jerrypeng commented on a change in pull request #3969: Cleanup temporary files
created by functions
URL: https://github.com/apache/pulsar/pull/3969#discussion_r271528751
##########
File path:
pulsar-broker/src/test/java/org/apache/pulsar/io/PulsarFunctionE2ETest.java
##########
@@ -450,6 +452,15 @@ 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"));
Review comment:
@merlimat yup I agree. That is why I have consolidated the code that
creates these temp files. This PR is already sizable so lets do this in a
subsequent PR
----------------------------------------------------------------
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]
With regards,
Apache Git Services
