[GitHub] flink issue #5059: [FLINK-8125] [core] Introduce limiting of file system con...

2017-11-24 Thread StefanRRichter
Github user StefanRRichter commented on the issue: https://github.com/apache/flink/pull/5059 LGTM 👍 ---

[GitHub] flink issue #5059: [FLINK-8125] [core] Introduce limiting of file system con...

2017-11-23 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5059 Addressed the comment. ---

[GitHub] flink issue #5059: [FLINK-8125] [core] Introduce limiting of file system con...

2017-11-23 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5059 It does not yet remove all checkstyle violation, no. ---

[GitHub] flink issue #5059: [FLINK-8125] [core] Introduce limiting of file system con...

2017-11-23 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/5059 Does this remove all checkstyle violations in core.fs? If so, please remove the exlcusion in `/tools/maven/suppressions-core.xml`. ``` ``` ---