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

Fix RapidsBufferStore NPE when no spillable buffers are available #928

Merged
merged 1 commit into from
Oct 12, 2020

Conversation

abellina
Copy link
Collaborator

Signed-off-by: Alessandro Bellina abellina@nvidia.com

Small bug that I observed could cause an NPE on OOM.

Signed-off-by: Alessandro Bellina <abellina@nvidia.com>
@abellina abellina added bug Something isn't working shuffle things that impact the shuffle plugin labels Oct 12, 2020
@abellina abellina requested a review from jlowe October 12, 2020 05:08
@abellina abellina changed the title RapidsBufferStore race could cause NPE Fix RapidsBufferStore NPE when no spillable buffers are available Oct 12, 2020
@revans2
Copy link
Collaborator

revans2 commented Oct 12, 2020

build

@jlowe jlowe merged commit 650e626 into NVIDIA:branch-0.3 Oct 12, 2020
sperlingxx pushed a commit to sperlingxx/spark-rapids that referenced this pull request Nov 20, 2020
Signed-off-by: Alessandro Bellina <abellina@nvidia.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Alessandro Bellina <abellina@nvidia.com>
nartal1 pushed a commit to nartal1/spark-rapids that referenced this pull request Jun 9, 2021
Signed-off-by: Alessandro Bellina <abellina@nvidia.com>
tgravescs pushed a commit to tgravescs/spark-rapids that referenced this pull request Nov 30, 2023
…IDIA#928)

Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working shuffle things that impact the shuffle plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants