Skip to content

Comments

[8.19] [EsAggs] Freeze cached responses to avoid response mutation (#222160)#226918

Merged
ppisljar merged 6 commits intoelastic:8.19from
ppisljar:backport/8.19/pr-222160
Jul 8, 2025
Merged

[8.19] [EsAggs] Freeze cached responses to avoid response mutation (#222160)#226918
ppisljar merged 6 commits intoelastic:8.19from
ppisljar:backport/8.19/pr-222160

Conversation

@ppisljar
Copy link
Contributor

@ppisljar ppisljar commented Jul 8, 2025

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

@ppisljar ppisljar added the backport This PR is a backport of another PR label Jul 8, 2025
@ppisljar ppisljar closed this Jul 8, 2025
@ppisljar ppisljar reopened this Jul 8, 2025
@ppisljar ppisljar marked this pull request as ready for review July 8, 2025 06:51
@ppisljar ppisljar requested a review from kibanamachine as a code owner July 8, 2025 06:51
@ppisljar ppisljar changed the title Backport/8.19/pr 222160 [8.19] [EsAggs] Freeze cached responses to avoid response mutation (#222160) Jul 8, 2025
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #26 / Security Solution - alerting api integration - security and spaces enabled Alerts - Group 1 alerts backfill rule runs ad hoc backfill task should handle timeouts

The CI Stats report is too large to be displayed here, check out the CI build annotation for this information.

History

Copy link
Contributor

@markov00 markov00 left a comment

Choose a reason for hiding this comment

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

Approve, not freezing the response, just cloning to avoid cache mutation

@ppisljar ppisljar merged commit 8a66504 into elastic:8.19 Jul 8, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants