-
Notifications
You must be signed in to change notification settings - Fork 66
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
Adding integ tests for scenario of hybrid query with aggregations #632
Adding integ tests for scenario of hybrid query with aggregations #632
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #632 +/- ##
=========================================
Coverage 82.62% 82.62%
Complexity 666 666
=========================================
Files 52 52
Lines 2072 2072
Branches 334 334
=========================================
Hits 1712 1712
Misses 212 212
Partials 148 148 ☔ View full report in Codecov by Sentry. |
0b4333e
to
a666584
Compare
Signed-off-by: Martin Gaievski <[email protected]>
d1f434d
to
78d0c2f
Compare
78d0c2f
to
edd02e3
Compare
...st/java/org/opensearch/neuralsearch/query/aggregation/BaseAggregationsWithHybridQueryIT.java
Outdated
Show resolved
Hide resolved
...st/java/org/opensearch/neuralsearch/query/aggregation/BaseAggregationsWithHybridQueryIT.java
Outdated
Show resolved
Hide resolved
.../java/org/opensearch/neuralsearch/query/aggregation/BucketAggregationsWithHybridQueryIT.java
Show resolved
Hide resolved
...st/java/org/opensearch/neuralsearch/query/aggregation/BaseAggregationsWithHybridQueryIT.java
Outdated
Show resolved
Hide resolved
edd02e3
to
2feadd2
Compare
…angelog Signed-off-by: Martin Gaievski <[email protected]>
2feadd2
to
19e2adf
Compare
dd3b30c
into
opensearch-project:main
* Adding tests and params to ignore tests if needed Signed-off-by: Martin Gaievski <[email protected]> (cherry picked from commit dd3b30c)
…ensearch-project#632) * Adding tests and params to ignore tests if needed Signed-off-by: Martin Gaievski <[email protected]> (cherry picked from commit dd3b30c) Signed-off-by: Martin Gaievski <[email protected]>
…) (#700) * Adding tests and params to ignore tests if needed (cherry picked from commit dd3b30c) Signed-off-by: Martin Gaievski <[email protected]> Co-authored-by: Martin Gaievski <[email protected]>
* Adding integ tests for scenario of hybrid query with aggregations (opensearch-project#632) * Adding tests and params to ignore tests if needed Signed-off-by: Martin Gaievski <[email protected]> * [BUG FIX] Fix bwc failure in neural sparse search (opensearch-project#696) * fix comments Signed-off-by: zhichao-aws <[email protected]> --------- Signed-off-by: Martin Gaievski <[email protected]> Signed-off-by: zhichao-aws <[email protected]> Co-authored-by: Martin Gaievski <[email protected]>
Description
Adding more integration tests for aggregations when they are used together with hybrid query. Picked 20 aggregations from official documentation.
Key points:
Example of test run:
Issues Resolved
#509
Check List
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.