>The actual field is never used but it will be used to check the allowed options when verifying what users specify via "string" options.
Are you saying that this option does get passed along to Elasticsearch still or that it's just arbitrarily validated? According to [1] it's been deprecated in ES 6 and removed in ES 7. [1] https://github.com/elastic/elasticsearch/pull/38085 On Wed, Jan 13, 2021 at 12:50 AM Aljoscha Krettek <aljos...@apache.org> wrote: > On 2021/01/12 15:04, Rex Fenley wrote: > >[2] > > > https://github.com/apache/flink/blob/97bfd049951f8d52a2e0aed14265074c4255ead0/flink-connectors/flink-connector-elasticsearch-base/src/main/java/org/apache/flink/streaming/connectors/elasticsearch/table/ElasticsearchOptions.java#L131 > > > >Should [2] be removed? > > The link seems to not work anymore but I'm guessing you're referring to > `CONNECTION_MAX_RETRY_TIMEOUT_OPTION`. This is used in the > `*DynamicSinkFactory` classes, such as [1]. These can be used when > defining Table API/SQL sources using DDL or the programmatic API. The > actual field is never used but it will be used to check the allowed > options when verifying what users specify via "string" options. > > [1] > > https://github.com/apache/flink/blob/ee653778689023ddfdf007d5bde1daad8fbbc081/flink-connectors/flink-connector-elasticsearch7/src/main/java/org/apache/flink/streaming/connectors/elasticsearch/table/Elasticsearch7DynamicSinkFactory.java#L98 > -- Rex Fenley | Software Engineer - Mobile and Backend Remind.com <https://www.remind.com/> | BLOG <http://blog.remind.com/> | FOLLOW US <https://twitter.com/remindhq> | LIKE US <https://www.facebook.com/remindhq>
