damgad commented on pull request #12072: URL: https://github.com/apache/beam/pull/12072#issuecomment-649843232
@aaltay The cron version (already deployed on the nodes, today) is less strict. It executes once a week and deletes files that were not accessed during at least two days. That should be fine for most cases, and safer for the executing builds. Here, the mechanism is more powerful, on-demand, and configurable. By specifying the `unaccessed_for` to `0` you can even delete all the tmp files. I've also considered having only that job with a cron trigger along with the manual one. But that's less reliable. In that case, if the node was for some reason overloaded or builds were stuck the cleanup wouldn't happen. ---------------------------------------------------------------- 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]
