Skip to content

Fix calls to copy_bitmask to pass stream parameter#14158

Merged
rapids-bot[bot] merged 4 commits intorapidsai:branch-23.10from
davidwendt:copy-bitmask-stream
Sep 22, 2023
Merged

Fix calls to copy_bitmask to pass stream parameter#14158
rapids-bot[bot] merged 4 commits intorapidsai:branch-23.10from
davidwendt:copy-bitmask-stream

Conversation

@davidwendt
Copy link
Copy Markdown
Contributor

Description

Fixes a couple places where cudf::copy_bitmask was called instead of cudf::detail::copy_bitmask to pass the available stream (and mr) parameters.

Found while reviewing #14121
Reference: #14121 (comment)

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@davidwendt davidwendt added bug Something isn't working 3 - Ready for Review Ready for review by team libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change labels Sep 21, 2023
@davidwendt davidwendt requested a review from a team as a code owner September 21, 2023 17:57
@davidwendt davidwendt self-assigned this Sep 21, 2023
@davidwendt davidwendt requested a review from bdice September 22, 2023 01:11
@davidwendt
Copy link
Copy Markdown
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 98b1bc6 into rapidsai:branch-23.10 Sep 22, 2023
@davidwendt davidwendt deleted the copy-bitmask-stream branch September 22, 2023 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Ready for Review Ready for review by team bug Something isn't working libcudf Affects libcudf (C++/CUDA) code. non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants