Skip to content

[8.19] ESQL: Fix CCS exchange sink cleanup (#143325)#143436

Merged
elasticsearchmachine merged 1 commit intoelastic:8.19from
ebarlas:backport/8.19/pr-143325
Mar 3, 2026
Merged

[8.19] ESQL: Fix CCS exchange sink cleanup (#143325)#143436
elasticsearchmachine merged 1 commit intoelastic:8.19from
ebarlas:backport/8.19/pr-143325

Conversation

@ebarlas
Copy link
Copy Markdown
Contributor

@ebarlas ebarlas commented Mar 3, 2026

Backports the following commits to 8.19:

The cross-cluster exchange sink in ClusterComputeHandler was
not explicitly finished on normal completion. It relied on
task cancellation or the InactiveSinksReaper for cleanup,
unlike data node sinks which finish immediately via
completion listeners.

Add a finishSinkHandler call to the exchange sink's
completion listener in runComputeOnRemoteCluster, matching
the pattern used by DataNodeComputeHandler. Also add an
@after assertion to CrossClusterQueryIT to verify that all
exchange sinks across clusters are released within 5 seconds
of each test.
@ebarlas ebarlas added :Analytics/ES|QL AKA ESQL >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Mar 3, 2026
@elasticsearchmachine elasticsearchmachine merged commit 7a6b119 into elastic:8.19 Mar 3, 2026
28 checks passed
@ebarlas ebarlas deleted the backport/8.19/pr-143325 branch March 3, 2026 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.19.13

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants