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

Avoid running sanitizer on Java test designed to cause an error #15753

Merged
merged 3 commits into from
May 15, 2024

Conversation

jlowe
Copy link
Member

@jlowe jlowe commented May 14, 2024

Description

Fixes NVIDIA/spark-rapids-jni#2039. CudaTest#testCudaException causes the compute-sanitizer to fail the test because it (correctly) flags an invalid argument being passed to a CUDA runtime call. Updated the tagging for the test to avoid running it under the compute-sanitizer.

Checklist

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

@jlowe jlowe added bug Something isn't working tests Unit testing for project Java Affects Java cuDF API. non-breaking Non-breaking change labels May 14, 2024
@jlowe jlowe self-assigned this May 14, 2024
@jlowe jlowe requested a review from a team as a code owner May 14, 2024 22:14
@jlowe
Copy link
Member Author

jlowe commented May 15, 2024

/merge

@rapids-bot rapids-bot bot merged commit 0811523 into rapidsai:branch-24.06 May 15, 2024
69 checks passed
@jlowe jlowe deleted the java-sanitizer-test-fix branch May 15, 2024 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Java Affects Java cuDF API. non-breaking Non-breaking change tests Unit testing for project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] COMPUTE-SANITIZER due to "invalid argument" on CUDA API call to cudaFreeHost
3 participants