-
Notifications
You must be signed in to change notification settings - Fork 2.3k
[GRPC][Term-level queries] Implement GRPC Ids, Range, and Terms Set queries #19448
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
[GRPC][Term-level queries] Implement GRPC Ids, Range, and Terms Set queries #19448
Conversation
Signed-off-by: Karen X <[email protected]>
|
❌ Gradle check result for 8dbf346: 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? |
Signed-off-by: Karen X <[email protected]>
8dbf346 to
b7abcfb
Compare
Signed-off-by: Karen X <[email protected]>
b7abcfb to
d89048d
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #19448 +/- ##
============================================
+ Coverage 72.88% 72.96% +0.07%
- Complexity 70219 70342 +123
============================================
Files 5702 5708 +6
Lines 322202 322392 +190
Branches 46625 46686 +61
============================================
+ Hits 234832 235226 +394
+ Misses 68418 68217 -201
+ Partials 18952 18949 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
...java/org/opensearch/transport/grpc/proto/request/search/query/IdsQueryBuilderProtoUtils.java
Outdated
Show resolved
Hide resolved
...main/java/org/opensearch/transport/grpc/proto/request/search/InnerHitsBuilderProtoUtils.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
a44d86c to
08037b3
Compare
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
|
❌ Gradle check result for 70eb7b3: 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? |
Signed-off-by: Karen X <[email protected]>
|
❌ Gradle check result for 24c230d: 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? |
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
f9228ff to
1fb2f8d
Compare
|
❌ Gradle check result for 1fb2f8d: 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? |
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
|
❌ Gradle check result for 98cafd2: 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? |
Signed-off-by: Karen X <[email protected]>
Signed-off-by: Karen X <[email protected]>
…ueries (opensearch-project#19448) * upgrade to protobufs 0.18.0 Signed-off-by: Karen X <[email protected]> * add tests Signed-off-by: Karen X <[email protected]> * [GRPC][Term-level queries] Implement Ids, Range, Termsset Signed-off-by: Karen X <[email protected]> * package private Signed-off-by: Karen X <[email protected]> * remove unused variables Signed-off-by: Karen X <[email protected]> * spotless Signed-off-by: Karen X <[email protected]> * remove duplicate from merge conflict Signed-off-by: Karen X <[email protected]> * spotless and fix merge conflict Signed-off-by: Karen X <[email protected]> * spotless Signed-off-by: Karen X <[email protected]> * fix merge conflict and spotless Signed-off-by: Karen X <[email protected]> --------- Signed-off-by: Karen X <[email protected]> Co-authored-by: karenx <[email protected]>
Description
Stacked on top of #19447
Implement GRPC Ids, Range, and Terms Set queries
Related Issues
Resolves #18907
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.