Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] Fix match_phrase_prefix_query not working on text field with multiple values and index_prefixes #14686

Merged
merged 1 commit into from
Jul 9, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 2e63913 from #10959.

… values and index_prefixes (#10959)

* Fix match_phrase_prefix_query not working on text field with multiple values and index_prefixes
Signed-off-by: Gao Binlong <[email protected]>

* Add more test

Signed-off-by: Gao Binlong <[email protected]>

* modify change log

Signed-off-by: Gao Binlong <[email protected]>

* Fix test failure

Signed-off-by: Gao Binlong <[email protected]>

* Change the indexAnalyzer used by prefix field

Signed-off-by: Gao Binlong <[email protected]>

* Skip old version for yaml test

Signed-off-by: Gao Binlong <[email protected]>

* Optimize some code

Signed-off-by: Gao Binlong <[email protected]>

* Fix test failure

Signed-off-by: Gao Binlong <[email protected]>

* Modify yaml test description

Signed-off-by: Gao Binlong <[email protected]>

* Remove the name parameter for setAnalyzer()

Signed-off-by: Gao Binlong <[email protected]>

---------

Signed-off-by: Gao Binlong <[email protected]>
(cherry picked from commit 2e63913)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented Jul 9, 2024

❌ Gradle check result for 297f2ba: 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?

Copy link
Contributor

github-actions bot commented Jul 9, 2024

✅ Gradle check result for 297f2ba: SUCCESS

Copy link

codecov bot commented Jul 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.46%. Comparing base (3fde6aa) to head (297f2ba).

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #14686      +/-   ##
============================================
- Coverage     71.52%   71.46%   -0.07%     
+ Complexity    62428    62343      -85     
============================================
  Files          5103     5103              
  Lines        293265   293268       +3     
  Branches      42730    42730              
============================================
- Hits         209766   209573     -193     
- Misses        65901    66037     +136     
- Partials      17598    17658      +60     

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

@reta reta merged commit 7040df2 into 2.x Jul 9, 2024
52 checks passed
kkewwei pushed a commit to kkewwei/OpenSearch that referenced this pull request Jul 24, 2024
… values and index_prefixes (opensearch-project#10959) (opensearch-project#14686)

* Fix match_phrase_prefix_query not working on text field with multiple values and index_prefixes

* Add more test

* modify change log

* Fix test failure

* Change the indexAnalyzer used by prefix field

* Skip old version for yaml test

* Optimize some code

* Fix test failure

* Modify yaml test description

* Remove the name parameter for setAnalyzer()

---------

(cherry picked from commit 2e63913)

Signed-off-by: Gao Binlong <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: kkewwei <[email protected]>
@dbwiddis dbwiddis deleted the backport/backport-10959-to-2.x branch July 24, 2024 16:04
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.

1 participant