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

Add RapidsHostMemoryStore tests #259

Merged
merged 2 commits into from
Jun 24, 2020
Merged

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Jun 22, 2020

Adds tests for the host buffer memory store.

@jlowe jlowe added the test Only impacts tests label Jun 22, 2020
@jlowe jlowe added this to the Jun 22 - Jul 2 milestone Jun 22, 2020
@jlowe jlowe self-assigned this Jun 22, 2020
revans2
revans2 previously approved these changes Jun 23, 2020
Copy link
Collaborator

@revans2 revans2 left a comment

Choose a reason for hiding this comment

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

Just one nit, that is not a requirement.

abellina
abellina previously approved these changes Jun 23, 2020
@jlowe jlowe dismissed stale reviews from abellina and revans2 via d1dd2f9 June 23, 2020 15:02
@revans2
Copy link
Collaborator

revans2 commented Jun 24, 2020

build

@revans2 revans2 merged commit 2cd8b36 into NVIDIA:branch-0.2 Jun 24, 2020
@jlowe jlowe deleted the hoststore-tests branch October 16, 2020 20:29
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Add RapidsHostMemoryStore tests

* Add comments about memory store taking ownership
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
* Add RapidsHostMemoryStore tests

* Add comments about memory store taking ownership
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
Signed-off-by: spark-rapids automation <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Only impacts tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants