Skip to content

[ESQL] Fix docs vector memory leak #137287

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
mouhc1ne merged 1 commit into from
Oct 28, 2025
Merged

[ESQL] Fix docs vector memory leak #137287

mouhc1ne merged 1 commit into from
Oct 28, 2025

Conversation

@mouhc1ne
Copy link
Contributor

@mouhc1ne mouhc1ne commented Oct 28, 2025 •
edited by dnhatn
Loading

Closes #136671

@mouhc1ne mouhc1ne requested a review from dnhatn October 28, 2025 18:41
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.3.0 labels Oct 28, 2025
@mouhc1ne mouhc1ne added >bug :Analytics/EQL EQL querying Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) and removed needs:triage Requires assignment of a team area label v9.3.0 labels Oct 28, 2025
@mouhc1ne mouhc1ne self-assigned this Oct 28, 2025
@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Oct 28, 2025

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

@mouhc1ne
Copy link
Contributor Author

mouhc1ne commented Oct 28, 2025 •
edited
Loading

Thanks @dnhatn for, well, fixing this! 🥇

@dnhatn dnhatn added the v9.3.0 label Oct 28, 2025
dnhatn
dnhatn approved these changes Oct 28, 2025
View reviewed changes
Copy link
Member

@dnhatn dnhatn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks @mouhc1ne. Maybe update the PR title?

@mouhc1ne mouhc1ne changed the title [CI] EnrichIT class failing [ESQL] Fix docs vector memory leak Oct 28, 2025
@mouhc1ne mouhc1ne added >test Issues or PRs that are addressing/adding tests and removed v9.2.1 labels Oct 28, 2025
@dnhatn dnhatn removed the >bug label Oct 28, 2025
@mouhc1ne mouhc1ne merged commit c3ee021 into elastic:main Oct 28, 2025
33 of 34 checks passed
chrisparrinello pushed a commit to chrisparrinello/elasticsearch that referenced this pull request Nov 3, 2025
@mouhc1ne @chrisparrinello
[CI] EnrichIT class failing (elastic#137287)
d1aef2c 
- Fix docs vector memory leak
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dnhatn dnhatn dnhatn approved these changes

Assignees

@mouhc1ne mouhc1ne

Labels

:Analytics/EQL EQL querying Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.3.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[CI] EnrichIT class failing

3 participants

@mouhc1ne @elasticsearchmachine @dnhatn