[GitHub] flink issue #3869: [FLINK-6502] Add support ElasticsearchSink for DataSet

2017-06-29 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3869 @397090770 I'm afraid I will need to reject this PR as of its current state. The main issue is that we can't just move those public user-facing classes. We either have to deprecate them and

[GitHub] flink issue #3869: [FLINK-6502] Add support ElasticsearchSink for DataSet

2017-05-18 Thread 397090770
Github user 397090770 commented on the issue: https://github.com/apache/flink/pull/3869 @tzulitai Thank you for your review very much. The current structuring of the ES output format as follow: ![elasticsearch_flink](https://cloud.githubusercontent.com/assets/5170878/26194858/69c