-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[8.x] fix: remove the _score field (#197197) #197561
Conversation
Fixes elastic#189624 Before: ![Pasted Graphic 1](https://github.com/user-attachments/assets/7669c66a-339c-40a5-8a20-8d6ca452612c) After: ![Pasted Graphic](https://github.com/user-attachments/assets/d4fa2ac6-9be1-4c77-8976-1df568b8aa7a) --------- Signed-off-by: fangshun <[email protected]> Co-authored-by: Davis McPhee <[email protected]> (cherry picked from commit 7dd66b3)
❌ Author of the following commits did not sign a Contributor Agreement: Please, read and sign the above mentioned agreement if you want to contribute to this project |
@elasticmachine merge upstream |
Strange, according to #197197 (comment) the contributor was signed. @fangshun-z was there a change from your end that could lead to complaining the cla checker again? |
I haven't had any changes on my end, but I just re-signed the agreement. Could you try triggering the process again? |
sure, thx! |
/ci |
dear @elastic/kibana-operations , do you have an idea, why the CLA check doesn't work here, thx! |
💚 Build Succeeded
Metrics [docs]Async chunks
History
|
Pinging @elastic/kibana-data-discovery (Team:DataDiscovery) |
@fangshun-z thanks a lot for the contribution and your patience! It looks like GitHub swapped your email address with their noreply email when this commit was merged - https://github.com/elastic/kibana/commit/7dd66b3fd97809ce563f323397fc810c2ac21d1a.patch. Our automation is cherry-picking that commit and pulling in an email that hasn't signed the CLA. Would you be able to open a pull request with these changes targeting the 8.x branch? Something like:
|
As you mentioned, I made modifications in this PR. |
Now that #198101 is merged, I think we're good to close this one. Thanks for opening up the other backport PR to get this in 👍 |
Pull request was closed
Backport
This will backport the following commits from
main
to8.x
:Questions ?
Please refer to the Backport tool documentation