JingsongLi opened a new pull request, #387:
URL: https://github.com/apache/paimon-rust/pull/387

   ## Summary
   
   Stabilizes Paimon's Vortex async IO by using a shared multi-thread Tokio 
runtime instead of binding each Vortex session to the caller's current Tokio 
runtime. This should address Windows CI hangs where concurrent Vortex IO tests 
could outlive or conflict with per-test Tokio runtimes.
   
   ## Changes
   
   - Keep a process-wide Vortex runtime alive for all Vortex reader/writer 
sessions.
   - Build Vortex sessions from that stable runtime handle instead of 
`Handle::try_current()`.
   - Add a regression test proving a Vortex runtime remains usable after the 
calling Tokio runtime is dropped.
   
   ## Testing
   
   - [x] `cargo fmt --all -- --check`
   - [x] `cargo test -p paimon --features fulltext,vortex 
arrow::format::vortex::tests:: -- --nocapture`
   - [x] `cargo test -p paimon --all-targets --features fulltext,vortex`
   
   ## Notes
   
   This does not skip any Vortex tests. The new regression test fails on the 
previous implementation with `Runtime dropped task without completing it`, then 
passes once Vortex IO is backed by the shared runtime.
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to