Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use stream in Java API. #11601

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Aug 25, 2022

Description

This fixes an issue where the Java API is using a default stream instead of the stream passed in. This is a small change separated from #11600 because it only touches one Java file and thus only needs a Java reviewer.

Checklist

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

@bdice bdice added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Aug 25, 2022
@bdice bdice self-assigned this Aug 25, 2022
@bdice bdice requested a review from a team as a code owner August 25, 2022 18:57
@bdice bdice added this to PR-WIP in v22.10 Release via automation Aug 25, 2022
@github-actions github-actions bot added the Java Affects Java cuDF API. label Aug 25, 2022
@bdice bdice added bug Something isn't working and removed bug Something isn't working labels Aug 25, 2022
@codecov
Copy link

codecov bot commented Aug 25, 2022

Codecov Report

❗ No coverage uploaded for pull request base (branch-22.10@5ee4b3b). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             branch-22.10   #11601   +/-   ##
===============================================
  Coverage                ?   86.41%           
===============================================
  Files                   ?      145           
  Lines                   ?    22959           
  Branches                ?        0           
===============================================
  Hits                    ?    19839           
  Misses                  ?     3120           
  Partials                ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

v22.10 Release automation moved this from PR-WIP to PR-Reviewer approved Aug 25, 2022
@bdice
Copy link
Contributor Author

bdice commented Aug 25, 2022

@gpucibot merge

@rapids-bot rapids-bot bot merged commit ae8e1df into rapidsai:branch-22.10 Aug 25, 2022
v22.10 Release automation moved this from PR-Reviewer approved to Done Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function Java Affects Java cuDF API. non-breaking Non-breaking change
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants