Skip to content
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

Fix potential block leak in LuceneSourceOperator #123835

Merged
merged 1 commit into from
Mar 3, 2025

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Mar 3, 2025

A follow-up to #123296 to address a potential block leak that may occur when a circuit-breaking exception is triggered while truncating the docs or scores blocks.

Relates #123296

@dnhatn dnhatn added >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport Automatically create backport pull requests when merged :Analytics/ES|QL AKA ESQL v8.18.1 v8.19.0 v9.0.1 v9.1.0 labels Mar 3, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 3, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine elasticsearchmachine merged commit 7560e2e into elastic:main Mar 3, 2025
17 checks passed
@dnhatn dnhatn deleted the fix-breaker branch March 3, 2025 02:31
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
8.18 Commit could not be cherrypicked due to conflicts
8.x Commit could not be cherrypicked due to conflicts
9.0 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 123835

dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Mar 3, 2025
A follow-up to elastic#123296 to address a potential block leak that may occur
when a circuit-breaking exception is triggered while truncating the docs
or scores blocks.

Relates  elastic#123296

(cherry picked from commit 7560e2e)
@dnhatn
Copy link
Member Author

dnhatn commented Mar 3, 2025

💚 All backports created successfully

Status Branch Result
8.x
9.0
8.18

Questions ?

Please refer to the Backport tool documentation

dnhatn added a commit to dnhatn/elasticsearch that referenced this pull request Mar 3, 2025
A follow-up to elastic#123296 to address a potential block leak that may occur
when a circuit-breaking exception is triggered while truncating the docs
or scores blocks.

Relates  elastic#123296

(cherry picked from commit 7560e2e)
dnhatn added a commit that referenced this pull request Mar 3, 2025
A follow-up to #123296 to address a potential block leak that may occur
when a circuit-breaking exception is triggered while truncating the docs
or scores blocks.

Relates  #123296

(cherry picked from commit 7560e2e)
dnhatn added a commit that referenced this pull request Mar 3, 2025
A follow-up to #123296 to address a potential block leak that may occur
when a circuit-breaking exception is triggered while truncating the docs
or scores blocks.

Relates  #123296

(cherry picked from commit 7560e2e)
dnhatn added a commit that referenced this pull request Mar 3, 2025
A follow-up to #123296 to address a potential block leak that may occur
when a circuit-breaking exception is triggered while truncating the docs
or scores blocks.

Relates  #123296
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.18.1 v8.19.0 v9.0.1 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants