Refresh property needs to be update to enum

[ejsmith]

I've noticed that the following 2 classes have a boolean Refresh property instead of using the Refresh enum:

DeleteByQueryRequest
UpdateByQueryRequest

[russcam]

thanks for reporting @ejsmith. They're both bool in the rest specs

• Update by query
• Delete by query

checking if these should be updated.

[Mpdreamz]

https://github.com/elastic/elasticsearch/blob/master/modules/reindex/src/main/java/org/elasticsearch/index/reindex/AbstractBaseReindexRestHandler.java#L94

The spec is correct here, thanks for reporting though @ejsmith 👍

[ejsmith]

Ok, thanks for checking and sorry for the trouble.

"Sending the refresh will update all shards in the index being updated when the request completes. This is different than the Index API’s refresh parameter which causes just the shard that received the new data to be indexed."