[ https://issues.apache.org/jira/browse/BEAM-8338?focusedWorklogId=340742&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-340742 ]
ASF GitHub Bot logged work on BEAM-8338: ---------------------------------------- Author: ASF GitHub Bot Created on: 08/Nov/19 20:42 Start Date: 08/Nov/19 20:42 Worklog Time Spent: 10m Work Description: regfaker commented on issue #10025: [BEAM-8338] Support ES 7.x for ElasticsearchIO URL: https://github.com/apache/beam/pull/10025#issuecomment-551982080 @echauchot At rest level (I mean requests) there are no breaking changes. I found couple of them at RestClient level itself when I upgraded es client dependency and that is what I'm working on now (almost finished). But at all I don't like this concept of ifing several versions in one class is good. Wouldn't it be better to have multiple ElasticsearchIO clients (per version) and ElasticsearchIO will use the one it needs (this will need some shading or if it is some other possibility in gradle (I'm not familiar with it)). I think that it is not sustainable in this state (for example if some big breaking change in RestClient will come and some of older versions (v2) will stop work). ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 340742) Time Spent: 2h 40m (was: 2.5h) > Support ES 7.x for ElasticsearchIO > ---------------------------------- > > Key: BEAM-8338 > URL: https://issues.apache.org/jira/browse/BEAM-8338 > Project: Beam > Issue Type: Improvement > Components: io-java-elasticsearch > Reporter: Michal BrunĂ¡t > Priority: Major > Time Spent: 2h 40m > Remaining Estimate: 0h > > Elasticsearch has released 7.4 but ElasticsearchIO only supports 2x,5.x,6.x. > We should support ES 7.x for ElasticsearchIO. > [https://www.elastic.co/guide/en/elasticsearch/reference/current/index.html] > > [https://github.com/apache/beam/blob/master/sdks/java/io/elasticsearch/src/main/java/org/apache/beam/sdk/io/elasticsearch/ElasticsearchIO.java] -- This message was sent by Atlassian Jira (v8.3.4#803005)