[GitHub] flink issue #3216: [FLINK-5637] Avoid warning while parsing YAML configurati...

2017-01-25 Thread rmetzger
Github user rmetzger commented on the issue: https://github.com/apache/flink/pull/3216 Yes. But I think it would be an overkill to add a test to ensure a certain log level. I'm going to merge the change now. --- If your project is set up for it, you can reply to this email a

[GitHub] flink issue #3216: [FLINK-5637] Avoid warning while parsing YAML configurati...

2017-01-25 Thread aljoscha
Github user aljoscha commented on the issue: https://github.com/apache/flink/pull/3216 Looks good, though we don't really test whether it silently ignores the empty lines, do we? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as