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.

Sign up for GitHub

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

Jump to bottom

Make DefaultHostMemoryAllocator settable #14523

Merged
merged 1 commit into from
Nov 29, 2023
Merged

Make DefaultHostMemoryAllocator settable #14523

merged 1 commit into from
Nov 29, 2023

Conversation

gerashegalov
Copy link
Contributor

@gerashegalov gerashegalov commented Nov 28, 2023 •
edited
Loading

spark-rapids requires the ability to set DefaultHostMemoryAllocator

Contributes to NVIDIA/spark-rapids#9862

Description

Checklist

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

@gerashegalov
Make DefaultHostMemoryAllocator settable
3eeec1b 
spark-rapids requires the ability to set DefaultHostMemoryAllocator

Contributes to NVIDIA/spark-rapids#9862

Signed-off-by: Gera Shegalov <[email protected]>
@gerashegalov gerashegalov requested a review from a team as a code owner November 28, 2023 22:46
@github-actions github-actions bot added the Java Affects Java cuDF API. label Nov 28, 2023
@gerashegalov gerashegalov self-assigned this Nov 28, 2023
@gerashegalov gerashegalov added 3 - Ready for Review Ready for review by team 4 - Needs cuDF (Java) Reviewer non-breaking Non-breaking change Spark Functionality that helps Spark RAPIDS feature request New feature or request labels Nov 28, 2023
jbrennan333
jbrennan333 approved these changes Nov 29, 2023
View reviewed changes
Copy link
Contributor

@jbrennan333 jbrennan333 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@gerashegalov
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit e696941 into rapidsai:branch-24.02 Nov 29, 2023
67 checks passed
@gerashegalov gerashegalov deleted the setDefaultHMA branch November 29, 2023 15:10
@vyasr vyasr added 4 - Needs Review Waiting for reviewer to review or respond and removed 4 - Needs cuDF (Java) Reviewer labels Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jbrennan333 jbrennan333 jbrennan333 approved these changes

@revans2 revans2 revans2 approved these changes

Assignees

@gerashegalov gerashegalov

Labels
3 - Ready for Review Ready for review by team 4 - Needs Review Waiting for reviewer to review or respond feature request New feature or request Java Affects Java cuDF API. non-breaking Non-breaking change Spark Functionality that helps Spark RAPIDS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gerashegalov @revans2 @jbrennan333 @vyasr