Skip to content

[8.x] Revert "Add ResolvedExpression wrapper (#114592)" (#115317) #115376

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 23, 2024
Merged

[8.x] Revert "Add ResolvedExpression wrapper (#114592)" (#115317) #115376

merged 1 commit into from
Oct 23, 2024

Conversation

@dakrone
Copy link
Member

@dakrone dakrone commented Oct 22, 2024

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

@original-brownbear
Revert "Add ResolvedExpression wrapper (elastic#114592)" (elastic#115317
d375002 
)

This reverts commit 4c15cc0.
This commit introduced an orders of magnitude regression when searching many shards.

(cherry picked from commit d9baf6f)
@dakrone dakrone mentioned this pull request Oct 22, 2024
Merged
@dakrone dakrone added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Data Management/Indices APIs APIs to create and manage indices and templates and removed v8.17.0 labels Oct 22, 2024
@elasticsearchmachine elasticsearchmachine merged commit 10f711d into elastic:8.x Oct 23, 2024
15 checks passed
@dakrone dakrone deleted the backport/8.x/pr-115317 branch October 23, 2024 00:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Data Management/Indices APIs APIs to create and manage indices and templates

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dakrone @elasticsearchmachine @original-brownbear