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 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
`flink-s3-fs-hadoop
Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/5072
Merging this...
---
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 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 fro
Github user greghogan commented on the issue:
https://github.com/apache/flink/pull/5072
@yew1eb have you looked at FLINK-6965?
---
