[Backport 2.x] 13776: allow adding query parameters to RequestOptions

[oliverlockwood]

Description

Allows adding query parameters to RequestOptions, e.g. to be able to set ?filter-path=<something> as documented in https://opensearch.org/docs/2.3/opensearch/common-parameters/#filtered-responses.

This is the same PR as #13777, but targeting the 2.x branch.

Related Issues

Resolves #13776.

Check List

• New functionality includes testing.
  • All tests pass
• New functionality has been documented.
  • New functionality has javadoc added
• API changes companion pull request created.
• Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
• Commits are signed per the DCO using --signoff
• Commit changes are listed out in CHANGELOG.md file (See: Changelog)
• Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[github-actions]

❌ Gradle check result for 9384163: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for fc0d014: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

✅ Gradle check result for abc8a8a: SUCCESS

[codecov]

Codecov Report

Attention: Patch coverage is 52.38095% with 10 lines in your changes missing coverage. Please review.

Project coverage is 71.36%. Comparing base (0dd892c) to head (8eb94f4).
Report is 298 commits behind head on 2.x.

Files	Patch %	Lines
...ain/java/org/opensearch/client/RequestOptions.java	62.50%	5 Missing and 1 partial ⚠️
...t/src/main/java/org/opensearch/client/Request.java	20.00%	3 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x   #13778      +/-   ##
============================================
+ Coverage     71.28%   71.36%   +0.07%     
- Complexity    60145    61359    +1214     
============================================
  Files          4957     5040      +83     
  Lines        282799   288500    +5701     
  Branches      41409    42148     +739     
============================================
+ Hits         201591   205875    +4284     
- Misses        64189    65277    +1088     
- Partials      17019    17348     +329     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[reta]

@oliverlockwood could you please rebase against latest 2.x? thank you.

[oliverlockwood]

@oliverlockwood could you please rebase against latest 2.x? thank you.

@reta I have now rebased this off the latest 2.x, squashed all my changes into one commit, and fixed the CHANGELOG.md conflict.

Please let me know if anything more is needed. Thanks.

[github-actions]

❌ Gradle check result for 8eb94f4: ABORTED

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[github-actions]

❌ Gradle check result for 8eb94f4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[reta]

❌ Gradle check result for 8eb94f4: FAILURE

#11179

[github-actions]

❌ Gradle check result for 8eb94f4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[reta]

❌ Gradle check result for 8eb94f4: FAILURE

#13433
#13600

[github-actions]

❌ Gradle check result for 8eb94f4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[reta]

❌ Gradle check result for 8eb94f4: FAILURE

#13695

[github-actions]

❌ Gradle check result for 8eb94f4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[reta]

❌ Gradle check result for 8eb94f4: FAILURE

#13600
#11179

[github-actions]

❌ Gradle check result for 8eb94f4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[reta]

❌ Gradle check result for 8eb94f4: FAILURE

#13828
#13473

[github-actions]

❌ Gradle check result for 8eb94f4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[reta]

❌ Gradle check result for 8eb94f4: FAILURE

#11179

[github-actions]

✅ Gradle check result for 8eb94f4: SUCCESS