kalencaya commented on code in PR #2543:
URL:
https://github.com/apache/incubator-seatunnel/pull/2543#discussion_r958043155
##########
seatunnel-connectors/seatunnel-connectors-flink/seatunnel-connector-flink-elasticsearch7/src/main/java/org/apache/seatunnel/flink/elasticsearch/sink/ElasticsearchOutputFormat.java:
##########
@@ -64,27 +62,14 @@ public ElasticsearchOutputFormat(Config userConfig,
ElasticsearchSinkFunction<T>
@Override
public void configure(Configuration configuration) {
List<String> hosts = config.getStringList(HOSTS);
- Settings.Builder settings = Settings.builder();
-
- config.entrySet().forEach(entry -> {
Review Comment:
Elasticsearch will not support settings on rest client, neither 7.x
RestHighLevelClient or 8.x ElasticsearchClient, for what all them based on
RestClient doesn't support it.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
