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

[Upport main] Made hybrid search active by default by flipping feature flag default #292

Conversation

martin-gaievski
Copy link
Member

@martin-gaievski martin-gaievski commented Sep 7, 2023

Manual port of 174f2c9 from #274

@martin-gaievski martin-gaievski added v2.10.0 Issues targeting release v2.10.0 Enhancements Increases software capabilities beyond original client specifications skip-changelog labels Sep 7, 2023
@martin-gaievski martin-gaievski marked this pull request as ready for review September 7, 2023 05:22
@navneet1v
Copy link
Collaborator

@martin-gaievski can you fix the build issue too. I know this is a backport to main from 2.x or you want to take that fix change separatly?

@martin-gaievski
Copy link
Member Author

@martin-gaievski can you fix the build issue too. I know this is a backport to main from 2.x or you want to take that fix change separatly?

Yes, exactly. Actually I created PR for fixing build issues few days back: #285.

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
(cherry picked from commit 174f2c9)
@martin-gaievski martin-gaievski force-pushed the backport/backport-274-to-main branch from 1cabbe8 to 4cc0cdc Compare September 7, 2023 23:57
@martin-gaievski martin-gaievski changed the title [Backport main] Made hybrid search active by default by flipping feature flag default [Upport main] Made hybrid search active by default by flipping feature flag default Sep 7, 2023
@codecov
Copy link

codecov bot commented Sep 8, 2023

Codecov Report

Merging #292 (4cc0cdc) into main (5bf36ed) will not change coverage.
The diff coverage is 87.50%.

@@            Coverage Diff            @@
##               main     #292   +/-   ##
=========================================
  Coverage     85.77%   85.77%           
  Complexity      376      376           
=========================================
  Files            30       30           
  Lines          1104     1104           
  Branches        168      168           
=========================================
  Hits            947      947           
  Misses           83       83           
  Partials         74       74           
Files Changed Coverage Δ
...g/opensearch/neuralsearch/plugin/NeuralSearch.java 77.77% <85.71%> (ø)
...ch/neuralsearch/settings/NeuralSearchSettings.java 100.00% <100.00%> (ø)

📢 Have feedback on the report? Share it here.

@martin-gaievski martin-gaievski merged commit 8484be9 into opensearch-project:main Sep 8, 2023
17 checks passed
xinyual pushed a commit to zhichao-aws/neural-search that referenced this pull request Sep 22, 2023
…opensearch-project#292)

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
(cherry picked from commit 174f2c9)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancements Increases software capabilities beyond original client specifications skip-changelog v2.10.0 Issues targeting release v2.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants