You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "QuakeWang (via GitHub)" <gi...@apache.org> on 2026/08/04 02:11:40 UTC
[PR] test(vindex): make executor cancellation test deterministic [paimon-rust]
QuakeWang opened a new pull request, #667:
URL: https://github.com/apache/paimon-rust/pull/667
### Purpose
`cancelling_waiter_does_not_stop_submitted_task` waited for the releaser thread instead of the executor task itself. Under scheduling contention, it could read the completion flag before the task resumed, causing intermittent CI failures.
### Brief change log
- Replace the timing-based completion assertion with an explicit oneshot signal.
- Cancel the waiter before releasing the submitted task, then wait for actual task completion with a bounded timeout.
### Tests
- `cargo fmt --all -- --check`
- `cargo clippy --locked --all-targets --workspace --features fulltext,vortex -- -D warnings`
- `cargo test --locked -p paimon --all-targets --features fulltext,vortex`
### API and Format
No API or storage format changes.
### Documentation
Not required.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test(vindex): make executor cancellation test deterministic [paimon-rust]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #667:
URL: https://github.com/apache/paimon-rust/pull/667
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org