fix(colexec): honor cancellation in window pipelines#26080
Conversation
Poll query cancellation during long-running window frame evaluation and partition output so KILL QUERY and KILL CONNECTION can stop the pipeline promptly. Add deterministic cancellation and cleanup regressions. Fixes matrixorigin#26067
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
XuPeng-SH
left a comment
There was a problem hiding this comment.
Deep-reviewed at e6c5220. Cancellation polling covers the long-running window frame/value/order paths and partition output; error exits are reclaimed through the existing Reset/Free lifecycle, and the operator protocol remains intact. Verified the new cancellation tests repeatedly, both owning packages in full and under race, plus the compile integration dependency. CI is green. No blocking findings.
Merge Queue Status
This pull request spent 2 hours 3 minutes 28 seconds in the queue, including 1 hour 3 minutes 11 seconds running CI. Required conditions to merge
|
Poll query cancellation during long-running window frame evaluation and partition output so KILL QUERY and KILL CONNECTION can stop the pipeline promptly. Add deterministic cancellation and cleanup regressions.
Fixes #26067
What type of PR is this?
Which issue(s) this PR fixes:
issue #26067
What this PR does / why we need it:
fix(colexec): honor cancellation in window pipelines