[GitHub] flink issue #5072: [FLINK-7984][build] Bump snappy-java to 1.1.4

2018-02-05 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/5072 @yew1eb Could you close the PR? The issue was addressed in f1e4d25c11a678688064492d50ffad38c39ea877. ---

[GitHub] flink issue #5072: [FLINK-7984][build] Bump snappy-java to 1.1.4

2017-12-15 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5072 Sorry, actually, taking a step back: Can you check in which dependencies snappy is included in addition? Especially big packaged dependencies like `flink-s3-fs-presto` and

[GitHub] flink issue #5072: [FLINK-7984][build] Bump snappy-java to 1.1.4

2017-12-15 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5072 Merging this... ---

[GitHub] flink issue #5072: [FLINK-7984][build] Bump snappy-java to 1.1.4

2017-12-15 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5072 Bumping the version should be fine. We only need to make sure that wherever this is pulled, we promote it to a direct dependency such that dependency convergence works. ---

[GitHub] flink issue #5072: [FLINK-7984][build] Bump snappy-java to 1.1.4

2017-12-08 Thread yew1eb
Github user yew1eb commented on the issue: https://github.com/apache/flink/pull/5072 @greghogan, I saw the following in the `flink-core` POM: ``` org.xerial.snappy snappy-java ``` I think this PR just upgrades the snappy-java version

[GitHub] flink issue #5072: [FLINK-7984][build] Bump snappy-java to 1.1.4

2017-12-07 Thread greghogan
Github user greghogan commented on the issue: https://github.com/apache/flink/pull/5072 @yew1eb have you looked at FLINK-6965? ---