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 address and length for GDS reads/writes [skip ci] #8301

Merged
merged 2 commits into from
May 20, 2021

Conversation

rongou
Copy link
Contributor

@rongou rongou commented May 20, 2021

Since we want GDS reads/writes to be 4 KiB aligned, sometimes we can't use the DeviceMemoryBuffer as is and need to adjust the size written. This change makes the JNI APIs more flexible to accommodate those.

@rongou rongou added 3 - Ready for Review Ready for review by team Performance Performance related issue Java Affects Java cuDF API. Spark Functionality that helps Spark RAPIDS improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 20, 2021
@rongou rongou requested a review from jlowe May 20, 2021 20:17
@rongou rongou self-assigned this May 20, 2021
@rongou rongou requested a review from a team as a code owner May 20, 2021 20:17
@rongou
Copy link
Contributor Author

rongou commented May 20, 2021

@gpucibot merge

@rapids-bot rapids-bot bot merged commit 2a1075e into rapidsai:branch-21.06 May 20, 2021
@rongou rongou deleted the gds-custom-size branch November 23, 2021 17:24
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 improvement Improvement / enhancement to an existing function Java Affects Java cuDF API. non-breaking Non-breaking change Performance Performance related issue Spark Functionality that helps Spark RAPIDS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants