Skip to content

Add warning on scripted metric aggregation's intermediate state memory usage. #119379

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
Jan 28, 2025
Merged

Add warning on scripted metric aggregation's intermediate state memory usage. #119379

merged 1 commit into from
Jan 28, 2025

Conversation

ppf2
Copy link
Contributor

@ppf2 ppf2 commented Dec 31, 2024 •
edited
Loading

Add a warning on scripted metric aggregation's intermediate state memory usage.

@github-actions GitHub Actions
Copy link
Contributor

Documentation preview:

@elasticsearchmachine
Copy link
Collaborator

@ppf2 please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Dec 31, 2024
@ppf2 ppf2 requested review from not-napoleon and rjernst December 31, 2024 01:40
@ppf2 ppf2 added >docs General docs changes Team:Docs Meta label for docs team labels Dec 31, 2024
@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Dec 31, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

not-napoleon
not-napoleon approved these changes Jan 3, 2025
View reviewed changes
Copy link
Member

@not-napoleon not-napoleon left a comment

Choose a reason for hiding this comment

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

I think documenting this is better than not

@leemthompo leemthompo added auto-backport Automatically create backport pull requests when merged v8.16.0 v8.17.0 v8.18.0 labels Jan 28, 2025
@leemthompo leemthompo merged commit 38b0e92 into main Jan 28, 2025
8 checks passed
@leemthompo leemthompo deleted the ppf2-scripted-agg-metric-memory branch January 28, 2025 10:10
This was referenced Jan 28, 2025
Merged
Merged
Merged
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
✅ 8.16
✅ 8.17
✅ 8.x

ppf2 added a commit to ppf2/elasticsearch that referenced this pull request Jan 28, 2025
@ppf2
Add warning on scripted metric aggregation's intermediate state memor…
2bf6be1 
…y usage (elastic#119379)
ppf2 added a commit to ppf2/elasticsearch that referenced this pull request Jan 28, 2025
@ppf2
Add warning on scripted metric aggregation's intermediate state memor…
fedbba2 
…y usage (elastic#119379)
elasticsearchmachine pushed a commit that referenced this pull request Jan 28, 2025
@ppf2
Add warning on scripted metric aggregation's intermediate state memor…
ae0bb70 
…y usage (#119379) (#121005)
elasticsearchmachine pushed a commit that referenced this pull request Jan 28, 2025
@ppf2
Add warning on scripted metric aggregation's intermediate state memor…
128d506 
…y usage (#119379) (#121004)
elasticsearchmachine pushed a commit that referenced this pull request Jan 28, 2025
@ppf2
Add warning on scripted metric aggregation's intermediate state memor…
e1c635b 
…y usage (#119379) (#121003)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@not-napoleon not-napoleon not-napoleon approved these changes

@rjernst rjernst Awaiting requested review from rjernst

Assignees

No one assigned

Labels

auto-backport Automatically create backport pull requests when merged >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Docs Meta label for docs team v8.16.0 v8.17.0 v8.18.0 v9.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ppf2 @elasticsearchmachine @not-napoleon @leemthompo