[GitHub] flink pull request: [FLINK-2061] CSVReader: quotedStringParsing an...

2015-05-27 Thread chiwanpark
GitHub user chiwanpark opened a pull request: https://github.com/apache/flink/pull/734 [FLINK-2061] CSVReader: quotedStringParsing and includeFields yields ParseException Fix the bug in `GenericCsvInputFormat` when skipped field is quoted string. I also added a unit test for this c

[GitHub] flink pull request: [FLINK-2061] CSVReader: quotedStringParsing an...

2015-05-28 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/734#issuecomment-106286147 Hi @chiwanpark , thanks for the PR! I'm a bit busy right now but will have a look at the PR soon. Thanks, Fabian --- If your project is set up for it, you can rep

[GitHub] flink pull request: [FLINK-2061] CSVReader: quotedStringParsing an...

2015-05-28 Thread chiwanpark
Github user chiwanpark commented on the pull request: https://github.com/apache/flink/pull/734#issuecomment-106313923 Okay. :) Because there is Stephen's email address in test code, I modified test code. --- If your project is set up for it, you can reply to this email and have yo

[GitHub] flink pull request: [FLINK-2061] CSVReader: quotedStringParsing an...

2015-05-29 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/734#issuecomment-106910782 Nice fix and good test case, thanks! Will run the tests and merge it. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] flink pull request: [FLINK-2061] CSVReader: quotedStringParsing an...

2015-05-29 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/734 --- 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 enabl