[GitHub] flink issue #4168: [FLINK-6987] Fix erroneous when path containing spaces

2017-06-27 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/4168 merging. --- 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 fea

[GitHub] flink issue #4168: [FLINK-6987] Fix erroneous when path containing spaces

2017-06-23 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4168 Thanks @twalthr for reporting this VERY good issue. And thanks @zentol for looking over it. PR has been updated. I misread the error from JIRA before , so literally used ```trim()``` to solve a

[GitHub] flink issue #4168: [FLINK-6987] Fix erroneous when path containing spaces

2017-06-22 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4168 Oops. I might be wrong. I will check again. --- 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 fea

[GitHub] flink issue #4168: [FLINK-6987] Fix erroneous when path containing spaces

2017-06-22 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4168 I dont think it has an underlying issue as below line of code throw an error, and it is not relavant to the ```TextInputFormat``` class . ```tmpDir``` has spaces and then trigger that problem.

[GitHub] flink issue #4168: [FLINK-6987] Fix erroneous when path containing spaces

2017-06-22 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/4168 I doubt this fixes the underlying issue, it just masks it by storing the input files in paths containing spaces. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] flink issue #4168: [FLINK-6987] Fix erroneous when path containing spaces

2017-06-22 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/4168 Hi @twalthr , you can test it again. I think it works fine now. --- 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 d