Skip to content

fix: keep Vortex runtime alive during async IO#375

Merged
JingsongLi merged 1 commit into
apache:mainfrom
QuakeWang:fix/vortex-linux-unit-hang
Jun 10, 2026
Merged

fix: keep Vortex runtime alive during async IO#375
JingsongLi merged 1 commit into
apache:mainfrom
QuakeWang:fix/vortex-linux-unit-hang

Conversation

@QuakeWang

Copy link
Copy Markdown
Contributor

Purpose

Fixes the Ubuntu unit test hang in Vortex format tests.

The previous code used VortexSession::default() directly. In Vortex 0.68, runtime handles are weak references to the underlying executor, so async read/write work can outlive the temporary runtime handle owner and stall when Vortex later spawns background IO tasks.

Brief change log

  • Create Vortex sessions from the active Tokio runtime.
  • Keep the Vortex runtime owner alive for the full reader stream lifetime.
  • Keep the Vortex runtime owner alive while the background writer task is running.

Tests

  • cargo fmt --all -- --check
  • git diff --check
  • cargo clippy --all-targets --workspace --features fulltext,vortex -- -D warnings
  • cargo test -p paimon --all-targets --features fulltext,vortex

API and Format

No API or storage format changes.

Documentation

No documentation update required.

Signed-off-by: QuakeWang <wangfuzheng0814@foxmail.com>

@JingsongLi JingsongLi 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.

+1

@JingsongLi JingsongLi merged commit d61fec5 into apache:main Jun 10, 2026
8 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