Fix extractArgument iterable casting to array

[rtyshyk]

Bulk API suports data as itarable, see \Elasticsearch\Endpoints\Bulk::setBody.

It has been broken during refactoring, see #56163ff.

[rtyshyk]

I signed, but not sure if / how to update CLA status in checks.

[elasticcla]

Hi @rtyshyk, we have found your signature in our records, but it seems like you have signed with a different e-mail than the one used in your Git commit. Can you please add both of these e-mails into your Github profile (they can be hidden), so we can match your e-mails to your Github profile?

[ezimuel]

@rtyshyk what is the issue that you are trying to solve? The Bulk endpoint supports Traversable (and also Iterator that extends Traversable) using this code.
The fact that Client::extractArgument() converts always an object into an array I don't think is an issue for the endpoint, even because this function is used also by other endpoints and not just Bulk. The Traversable support in Bulk endpoint has been added in #351.

[ezimuel]

I signed, but not sure if / how to update CLA status in checks.

The CLA signature is ok now.

[rtyshyk]

@ezimuel Yes, bulk endpoint supports Traversable, but the issue here is it is not possible to pass Traversable to it :)

I use \Elasticsearch\Client to consume bulk endpoint and pass there Iterator that returns right meta/data to build a body as well as reduce memory usage.

to make it simple

$client = new \Elasticsearch\Client()
$result = $client->bulk([
    'body' => new \IteratorIterator(new \ArrayIterator([1, 2, 3])),
]);

\Elasticsearch\Client::bulk uses extractArgument for the body parameter where is extractArgument do a cast any object to array.

Traversable cannot be cast to an array, as a result, there is an empty data argument.

https://3v4l.org/Ocq8P

[ezimuel]

@rtyshyk I see what you mean, I wrongly supposed that casting IteratorIterator into array was working fine. I'll merge this PR soon.
Thanks for your contribution!