ScriptFile parameter for NEST UpdateDescriptor

[vineet85]

Re: #1456

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

[gmarz]

Added! Nice catch @vineet85 and thanks again for reporing.

[vineet85]

Thanks @gmarz - Any idea when v1.5.2 will be publicly available?

[gmarz]

Very soon...debating whether to release 1.5.2 or go straight to 1.6 (pending #1453).

[CezarCretu]

This doesn't seem to be working for me using the newly-released 1.6.
I've checked the deserialized request body and the value set for ScriptFile isn't included.

Should ScriptFile and ScriptId not be added in BulkUpdateBody and be set here or am I just missing something?

[gmarz]

Argh, yes, you're right @CezarCretu. This is my bad...we'll fix and release a 1.6.1 asap.

[gmarz]

FYI I opened #1463 for this.