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

Fixed Hybrid query for cases when it's wrapped into other compound queries #498

Merged

Dec 1, 2023

Merged

Added comments, improved exception handling

Codecov / codecov/patch failed Nov 29, 2023 in 0s

54.54% of diff hit (target 84.65%)

54.54% of diff hit (target 84.65%)

src/main/java/org/opensearch/neuralsearch/search/query/HybridQueryPhaseSearcher.java#L105

Added line #L105 was not covered by tests

src/main/java/org/opensearch/neuralsearch/search/query/HybridQueryPhaseSearcher.java#L165-L166

Added lines #L165 - L166 were not covered by tests

src/main/java/org/opensearch/neuralsearch/search/query/HybridQueryPhaseSearcher.java#L179-L180

Added lines #L179 - L180 were not covered by tests

src/main/java/org/opensearch/neuralsearch/search/query/HybridQueryPhaseSearcher.java#L182

Added line #L182 was not covered by tests