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

Add RapidsBufferCatalog tests #193

Merged
merged 3 commits into from
Jun 17, 2020
Merged

Add RapidsBufferCatalog tests #193

merged 3 commits into from
Jun 17, 2020

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented Jun 16, 2020

Adds unit tests for RapidsBufferCatalog and removes the stores map within the catalog as it was effectively unused.

@jlowe jlowe added the shuffle things that impact the shuffle plugin label Jun 16, 2020
@jlowe jlowe added this to the Jun 22 - Jul 2 milestone Jun 16, 2020
@jlowe jlowe self-assigned this Jun 16, 2020
revans2
revans2 previously approved these changes Jun 17, 2020
View reviewed changes
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.

Needs to be upmerged

@revans2
Copy link
Collaborator

revans2 commented Jun 17, 2020

build

@jlowe jlowe merged commit 6e7fa77 into NVIDIA:branch-0.1 Jun 17, 2020
@jlowe jlowe deleted the buffer-catalog-tests branch June 17, 2020 16:59
@sameerz sameerz added the test Only impacts tests label Jul 3, 2020
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
@jlowe
Add RapidsBufferCatalog tests (NVIDIA#193)
eabf6d7
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
@jlowe
Add RapidsBufferCatalog tests (NVIDIA#193)
c2c014a
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
@nvauto
Update submodule cudf to 31a5f44 (NVIDIA#193)
c55d627 
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
Reviewers

@revans2 revans2 revans2 approved these changes

@abellina abellina abellina approved these changes

Assignees

@jlowe jlowe

Labels
shuffle things that impact the shuffle plugin test Only impacts tests
Projects
None yet
Milestone
Jun 22 - Jul 2
Development

Successfully merging this pull request may close these issues.
4 participants
@jlowe @revans2 @abellina @sameerz