[GitHub] flink issue #5243: [FLINK-8362][elasticsearch] shade all dependencies

2018-01-10 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/5243 I can't recall any license issues with shading the ES connectors ... will keep an extra eye and double check on that before merging this. @NicoK thanks a lot for your work on this. I'll

[GitHub] flink issue #5243: [FLINK-8362][elasticsearch] shade all dependencies

2018-01-05 Thread StephanEwen
Github user StephanEwen commented on the issue: https://github.com/apache/flink/pull/5243 I vaguely recall that there was a reason why we did not shade the Elasticsearch connectors initially. Maybe a license issue. @zentol and @tzulitai can you chime in here? If we want to