-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Added support for msearch API to pass search pipeline name #15923
base: main
Are you sure you want to change the base?
Conversation
❌ Gradle check result for 59c9cfd: 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? |
141883d
to
00df542
Compare
❌ Gradle check result for 141883d: 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? |
❌ Gradle check result for 00df542: 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? |
❌ Gradle check result for d2fe2fb: 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? |
d2fe2fb
to
87fcc40
Compare
❌ Gradle check result for 87fcc40: 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? |
❌ Gradle check result for 87fcc40: 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? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ahh! You lost the part that copies from the source into SearchRequest
on the regular _search
path. It would be really nice to get that part too.
Signed-off-by: Owais <[email protected]>
Signed-off-by: Owais <[email protected]>
Signed-off-by: Owais <[email protected]>
Signed-off-by: Owais <[email protected]>
Signed-off-by: Owais <[email protected]>
87fcc40
to
fe74c21
Compare
❌ Gradle check result for fe74c21: 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: Owais <[email protected]>
❌ Gradle check result for b2c2001: 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? |
server/src/main/java/org/opensearch/search/builder/SearchSourceBuilder.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Owais <[email protected]>
❌ Gradle check result for a1bc82b: 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? |
if (in.getVersion().onOrAfter(Version.V_2_18_0)) { | ||
searchPipeline = in.readOptionalString(); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll need to do the whole BWC dance, since 2.x is building 2.18.
- Do this on main with
onOrAfter(Version.V_3_0_0))
. Get it merged. - You'll need a manual backport to 2.x, where you do
onOrAfter(Version.V_2_18_0)
. Don't get it merged right away. - Before merging the backport to 2.x, open another PR on main to change it to
onOrAfter(Version.V_2_18_0)
. - Merge the backport PR.
- Merge the main version update PR.
We need to do those last two steps as close together as possible, as merging the backport PR will break builds on main.
I hate this process, but we haven't been able to think of something better.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, you are right. Been a while changed an API body. Pushed the change and also added one more test for serialization.
❌ Gradle check result for f64a7cb: 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? |
❌ Gradle check result for f64a7cb: 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: Owais <[email protected]>
f64a7cb
to
a6a64ee
Compare
❌ Gradle check result for a6a64ee: 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? |
❕ Gradle check result for a6a64ee: UNSTABLE Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #15923 +/- ##
============================================
+ Coverage 71.96% 71.98% +0.02%
- Complexity 64391 64418 +27
============================================
Files 5280 5280
Lines 300866 300886 +20
Branches 43474 43480 +6
============================================
+ Hits 216510 216589 +79
+ Misses 66639 66495 -144
- Partials 17717 17802 +85 ☔ View full report in Codecov by Sentry. |
Description
Added support for msearch API to pass search pipeline name.
With this change a search pipeline name can be provided while making a msearch call
Related Issues
Resolves #15748
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.