Skip to content

Replace libcudf cuda stream owners - #24234

Draft
vyasr wants to merge 1 commit into
NVIDIA:mainfrom
vyasr:codex/cuda-stream-migration
Draft

vyasr wants to merge 1 commit into
NVIDIA:mainfrom
vyasr:codex/cuda-stream-migration

Conversation

@vyasr

@vyasr vyasr commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace direct rmm::cuda_stream owners in libcudf tests and the FST benchmark with device-bound cuda::stream
  • retain explicit synchronization while adopting non-blocking cuda::stream creation
  • preserve the stream-pool examples and libcudf_streaming interfaces
  • update stream-identification documentation for its cuda::stream owner

Validation

  • built libcudf
  • rebuilt libcudf_streaming
  • passed focused AST, column rebind, scalar async, FST, logical-stack, streaming hash-join, and streaming groupby tests
  • compiled the modified FST benchmark translation unit
  • git diff --check
  • verified remaining rmm::cuda_stream references are only deferred rmm::cuda_stream_pool examples

Notes

The local pre-commit suite passed all relevant checks; its Lychee hook is currently misconfigured because the configured revision is not a versioned release tag.

@copy-pr-bot

copy-pr-bot Bot commented Sep 19, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@github-actions github-actions Bot added the libcudf Affects libcudf (C++/CUDA) code. label Sep 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

libcudf Affects libcudf (C++/CUDA) code.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant