Skip to content

fix(colexec): honor cancellation in window pipelines#26080

Merged
mergify[bot] merged 6 commits into
matrixorigin:mainfrom
jiangxinmeng1:agent/fix-window-union-cancel-26067
Jul 24, 2026
Merged

fix(colexec): honor cancellation in window pipelines#26080
mergify[bot] merged 6 commits into
matrixorigin:mainfrom
jiangxinmeng1:agent/fix-window-union-cancel-26067

Conversation

@jiangxinmeng1

Copy link
Copy Markdown
Contributor

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?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #26067

What this PR does / why we need it:

fix(colexec): honor cancellation in window pipelines

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-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@matrix-meow matrix-meow added the size/M Denotes a PR that changes [100,499] lines label Jul 23, 2026

@XuPeng-SH XuPeng-SH left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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.

@mergify mergify Bot added the queued label Jul 24, 2026
@mergify

mergify Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • Entered queue2026-07-24 07:20 UTC · Rule: main · triggered by rule Automatic queue on approval for main
  • Checks passed · in-place
  • Merged2026-07-24 09:24 UTC · at 0cd8caccc90020aa74e895285a2c9d0431898cc2 · squash

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
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • github-review-approved [🛡 GitHub branch protection] (documentation)
  • github-review-decision = APPROVED [🛡 GitHub branch protection] (documentation)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Compose CI / multi cn e2e bvt test docker compose(PESSIMISTIC)
    • check-neutral = Matrixone Compose CI / multi cn e2e bvt test docker compose(PESSIMISTIC)
    • check-skipped = Matrixone Compose CI / multi cn e2e bvt test docker compose(PESSIMISTIC)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Standlone CI / Multi-CN e2e BVT Test on Linux/x64(LAUNCH, PROXY)
    • check-neutral = Matrixone Standlone CI / Multi-CN e2e BVT Test on Linux/x64(LAUNCH, PROXY)
    • check-skipped = Matrixone Standlone CI / Multi-CN e2e BVT Test on Linux/x64(LAUNCH, PROXY)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH, PESSIMISTIC)
    • check-neutral = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH, PESSIMISTIC)
    • check-skipped = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH, PESSIMISTIC)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone CI / UT Test on Ubuntu/x86
    • check-neutral = Matrixone CI / UT Test on Ubuntu/x86
    • check-skipped = Matrixone CI / UT Test on Ubuntu/x86
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Utils CI / Coverage
    • check-neutral = Matrixone Utils CI / Coverage
    • check-skipped = Matrixone Utils CI / Coverage
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone CI / SCA Test on Linux/arm64
    • check-neutral = Matrixone CI / SCA Test on Linux/arm64
    • check-skipped = Matrixone CI / SCA Test on Linux/arm64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Something isn't working size/M Denotes a PR that changes [100,499] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: KILL QUERY / KILL CONNECTION does not cancel long-running Window+UNION query (session stays COM_QUERY for 40h+)

3 participants