Remove '| string' in query parameters

[pquentin]

The | string isn't meaningful in those two cases and this helps #5215. I'm skipping backports since this is a breaking change, though.

[pquentin]

While this one isn't a query parameter, it feels like a straight-up bug.

Parsing is done in https://github.com/elastic/elasticsearch/blob/eb3bc2f2929209461802884ea01f77d3b3fcf96a/x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/action/GetOverallBucketsAction.java#L73

[github-actions]

Following you can find the validation changes against the target branch for the API.

API	Status	Request	Response
cat.nodes	🟢 → 🔴	11/11 → 10/11	11/11

You can validate this API yourself by using the make validate target.

[pquentin]

At least I have the reason why this was done: YAML tests!

• full_id is sent as a boolean, but gets recorded as a string somehow: I attempted a fix in https://github.com/elastic/clients-flight-recorder/pull/1376
• overall_score is a YAML test bug: [ML] Fix YAML test to use correct query parameter type elasticsearch#134999

[davidkyle]

The ML change LGTM

[pquentin]

Merging! There's one failure left on cat.nodes, but it will be fixed with the next clients-flight-recorder run.