Script_File support for NEST

[clintongormley]

From @vineet85 on June 14, 2015 20:15

Hi,

ElasticSearch supports referencing a script which is stored in a file. Eg:-
GET /_search
{
"script_fields": {
"my_field": {
"script_file": "my_script",
"params": {
"my_var": 2
}
}
}
}

However the .NET NEST client does not have any support for this yet. Is this going to be added soon?

elasticsearch-net/src/Nest/DSL/Filter/ScriptFilterDescriptor.cs

Line 36 in 4fcecae

public string ScriptId { get; set; }

Thanks,
Vineet

Copied from original issue: elastic/elasticsearch#11652

[isabel12]

This would be great - it is best practice to use stored scripts over anything else isn't it? (https://www.elastic.co/guide/en/elasticsearch/reference/current/modules-scripting.html)

It would be nice to be able to use scripts safely and disable dynamic scripting.

[gmarz]

Just added. Thanks for reporting @vineet85

[vineet85]

Can this be added to the BulkUpdateDescriptor as well?

https://github.com/elastic/elasticsearch-net/blob/4fcecae93dcf16a4ff7e110135da81eed409dda5/src/Nest/DSL/Bulk/BulkUpdateDescriptor.cs

Eg:-
BulkDescriptor childBulkDescriptor = new BulkDescriptor();

for (int i = 0; i < searchResponse.Hits.Count(); i++)
{
childBulkDescriptor.Update<NestedDocument, object>(u => u

• .Type("document")*
• .Id(searchResponse.Hits.ElementAt(i).Id) // document id*
• .ScriptFile("my_script")*
• .Params(x => x.Add("field_id", fieldIdToUpdate).Add("field_value",
  fieldValue))*
• .RetriesOnConflict(4)*
• );*
• }*

Thanks,
Vineet

On Sun, Jun 14, 2015 at 10:16 PM, Greg Marzouka notifications@github.com
wrote:

Just added. Thanks for reporting @vineet85 https://github.com/vineet85

—
Reply to this email directly or view it on GitHub
#1456 (comment)
.

Vineet Saju