Skip to content

Conversation

@idegtiarenko
Copy link
Contributor

This change simplifies exchange sink cleanup after task cancelation handling by moving and grouping handlers together.

@idegtiarenko idegtiarenko requested review from dnhatn and nik9000 May 15, 2025 06:59
@idegtiarenko idegtiarenko added >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v9.1.0 labels May 15, 2025
@elasticsearchmachine
Copy link
Collaborator

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

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 @idegtiarenko

@idegtiarenko idegtiarenko merged commit e38bbc0 into elastic:main May 20, 2025
17 checks passed
@idegtiarenko idegtiarenko deleted the simplify_sink_release_logic branch May 20, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL >non-issue Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants