Update es read query to not use body

[dstandish]

The argument is deprecated and it's advised to use the params directly.

[jedcunningham]

When was it deprecated / when was params directly supported? Just trying to figure out if we have a new min version of ES we support or something.

[dstandish]

Yeah saw test failures. I'm seeing the deprecation at version 8.1

We have it pinned at >8, <9 it seems. marked draft for now

[eladkal]

cc @Owen-CH-Leung you mentioned the count api in https://github.com/apache/airflow/pull/33135/files#r1285877721

[Owen-CH-Leung]

Hmm Indeed. It looks like body param is getting deprecated soon. I was diving into the v8.10.0 library (the latest v8 one) here, and I believe it will print this warning message (but strange I didn't see this warning when I was doing the upgrade before / testing the remote logging capability )

https://github.com/elastic/elasticsearch-py/blob/v8.10.0/elasticsearch/_sync/client/utils.py#L393-L404

[dstandish]

I fixed (in this PR) the calls so that they work with 8.10...

I have a hunch that to fix the tests in this PR we'll need to bump es version to 8.10

[dstandish]

OK I tested this change locally and it does require bumping min elasticsearch version to 8.10 (which i have updated the provider settings to do)

So, presuming tests pass, this should be good to go

[potiuk]

Nice. I like when our provider's min versions of dependencies got update. I am actualy thinking of adding a check for it - i.e. attempting to install lowest versions of dependencies possible and running our tests on it. That would be an ulitmate check whether our min requirements

I captured it here #35549 and I will likely find some time to implement it in the coming months, though if someone else would like to pick it up, I am happy to guide them and work together.