Skip to content

test(vindex): make PK-vector concurrency tests deterministic - #685

Merged
JingsongLi merged 1 commit into
apache:mainfrom
QuakeWang:fix-ann-flake
Aug 6, 2026
Merged

test(vindex): make PK-vector concurrency tests deterministic#685
JingsongLi merged 1 commit into
apache:mainfrom
QuakeWang:fix-ann-flake

Conversation

@QuakeWang

@QuakeWang QuakeWang commented Aug 6, 2026

Copy link
Copy Markdown
Member

Purpose

PK-vector concurrency tests inferred task overlap from short sleeps and assumed the shared executor had enough physical workers. On constrained or busy CI runners, correct concurrent code could execute serially and fail nondeterministically.

Brief change log

  • Ensure the executor capacity required by parallel test cases.
  • Replace sleep-based overlap checks with bounded scorer gates.
  • Serialize gated tests and verify shared-budget permits and execution waves explicitly.

Tests

  • cargo fmt --all -- --check
  • cargo clippy --locked --all-targets --workspace --features fulltext,vortex -- -D warnings
  • cargo +1.91.0 check --locked --workspace --all-targets --all-features
  • cargo test --locked -p paimon --all-targets --features fulltext,vortex

API and Format

Documentation

Signed-off-by: QuakeWang <wangfuzheng0814@foxmail.com>
@JingsongLi
JingsongLi merged commit 83118a1 into apache:main Aug 6, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants