Github user zentol commented on the pull request:
https://github.com/apache/incubator-flink/pull/202#issuecomment-63980978
1. All files will be removed even if the job fails. (on workernodes,
master comes with next commit)
2. Why should they be moved from /tmp to the home directory? the distCache
puts files in /tmp, I'd rather have them all in roughly the same place.
3. No tests.
4. instead of making it a configurable value, let's just fix the code to
work under python 3. all relevant code bits are in the setup stage, as in
performance unrelated.
5. StringIO import is actually unused, look at that ^^
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
