elastic / elasticsearch-hadoop

:elephant: Elasticsearch real-time search and analytics natively integrated with Hadoop
https://www.elastic.co/products/hadoop
Apache License 2.0
1.93k stars 986 forks source link

Stop using deprecated "stored" field in scripts for updates #2213

Open masseyke opened 3 months ago

masseyke commented 3 months ago

As part of #918 and #538 we added the ability to call a stored script from an update. Shortly after that, the stored keyword was deprecated (https://github.com/elastic/elasticsearch/blob/main/server/src/main/java/org/elasticsearch/script/ScriptType.java#L42 was deprecated here https://github.com/elastic/elasticsearch/pull/25127). It looks like we need to be using id instead where we currently use stored here: https://github.com/elastic/elasticsearch-hadoop/blob/main/mr/src/main/java/org/elasticsearch/hadoop/serialization/bulk/UpdateBulkFactory.java#L82