Skip to content

fix: prioritize stalled checkpoint bounded flush#24933

Open
jiangxinmeng1 wants to merge 5 commits into
matrixorigin:4.0-devfrom
jiangxinmeng1:cherry-pick-24901-to-4.0-dev
Open

fix: prioritize stalled checkpoint bounded flush#24933
jiangxinmeng1 wants to merge 5 commits into
matrixorigin:4.0-devfrom
jiangxinmeng1:cherry-pick-24901-to-4.0-dev

Conversation

@jiangxinmeng1

@jiangxinmeng1 jiangxinmeng1 commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

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

Which issue(s) does this PR fix or relate to?

Fixes #24900

What this PR does / why we need it:

Cherry-pick changes from #24901 to 4.0-dev.

Picked commits:

  • d704100 fix: prioritize stalled checkpoint bounded flush
  • 9f546cf fix: keep bounded checkpoint flush requests isolated
  • 97872cc fix: preserve flush request time ranges
  • 6f9c2ba fix: bypass flush readiness for stalled checkpoints
  • 62ca6d8 fix: preserve deadline for bounded checkpoint flush

Local verification:

  • go test ./pkg/vm/engine/tae/db/checkpoint blocked by inconsistent vendor metadata in this checkout.
  • go test -mod=mod ./pkg/vm/engine/tae/db/checkpoint reached compilation but failed because local CGO headers are missing (usearch.h, xxhash.h).
  • CGO_ENABLED=0 go test -mod=mod ./pkg/vm/engine/tae/db/checkpoint is not supported because this package graph depends on CGO allocator/usearch bindings.

@jiangxinmeng1 jiangxinmeng1 requested a review from XuPeng-SH as a code owner June 11, 2026 06:17
@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 →

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/L Denotes a PR that changes [500,999] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants