Skip to content

Conversation

@karenyrx
Copy link
Collaborator

@karenyrx karenyrx commented Jul 17, 2025

Description

Fix MultiMatchQuery protos

  1. Renumber protos so required fields use smaller numbers
  2. Add optional to all optional fields
  3. Add comments
  4. Remove deprecated fields fuzziness and cutoff_frequency.

Spec/public doc issues to fix

  1. Fuzziness is deprecated 3 years ago: https://github.com/opensearch-project/OpenSearch/pull/5433/files
  2. cutoff_frequency is deprecated 6 years ago in ES 7: Deprecate CommonTermsQuery and cutoff_frequency elastic/elasticsearch#42619
    Will fix spec and public doc in a separate PR.

Issues Resolved

#30

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.

@karenyrx karenyrx force-pushed the multimatch branch 2 times, most recently from 86041e9 to 566443a Compare July 18, 2025 00:40
@karenyrx karenyrx marked this pull request as ready for review July 18, 2025 01:48
@karenyrx karenyrx merged commit ee1f4bd into opensearch-project:main Jul 18, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants