Skip to content

[9.1] [Streams 🌊] Prevent routing simulation timeout and improve cancellations (#226374)#227246

Merged
tonyghiani merged 4 commits intoelastic:9.1from
tonyghiani:backport/9.1/pr-226374
Jul 9, 2025
Merged

[9.1] [Streams 🌊] Prevent routing simulation timeout and improve cancellations (#226374)#227246
tonyghiani merged 4 commits intoelastic:9.1from
tonyghiani:backport/9.1/pr-226374

Conversation

@tonyghiani
Copy link
Contributor

Backport

This will backport the following commits from main to 9.1:

Questions ?

Please refer to the Backport tool documentation

…ons (elastic#226374)

## 📓 Summary

Closes elastic#219494

Refactor the samples fetching logic and address the following issues:
- Add a timeout (10 seconds) to avoid requests on incomplete or
unmatching conditions to keep running with no feedback.
- Update cancellation logic, previously was not catching correctly the
request abortions.
- Sample data fetching performed on the creation form only.

(cherry picked from commit 00333f2)

# Conflicts:
#	x-pack/platform/plugins/shared/streams_app/public/components/data_management/stream_detail_enrichment/processor_outcome_preview.tsx
@tonyghiani tonyghiani requested a review from kibanamachine as a code owner July 9, 2025 13:12
@tonyghiani tonyghiani added the backport This PR is a backport of another PR label Jul 9, 2025
@tonyghiani tonyghiani enabled auto-merge (squash) July 9, 2025 13:16
@tonyghiani tonyghiani merged commit 3cc44e5 into elastic:9.1 Jul 9, 2025
12 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
streamsApp 578.2KB 583.2KB +4.9KB
Unknown metric groups

ESLint disabled line counts

id before after diff
streamsApp 8 7 -1

Total ESLint disabled count

id before after diff
streamsApp 12 11 -1

@tonyghiani tonyghiani deleted the backport/9.1/pr-226374 branch February 20, 2026 12:05
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.

3 participants