-
Notifications
You must be signed in to change notification settings - Fork 240
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
Additional unit tests for GeneratedInternalRowToCudfRowIterator #10087
Conversation
Signed-off-by: Jim Brennan <[email protected]>
build |
Just FYI, some of the tests that were added with this PR are failing in the premerge tests. It appears to be for
|
build |
I am having trouble reproducing the premerge failure locally. I can step through the unit tests in the debugger, and the |
I figured out what was happening here. In premerge, another test ( |
build |
@revans2 this should be good now. |
build |
Adding some additional tests to
GeneratedInternalRowToCudfRowIteratorRetrySuite
.The existing GPU OOM retry tests were actually testing CPU ooms, after the CPU Host Memory retry changes went in.
Now that we have improved the forced OOM interfaces, we can fix these.
I changed the existing tests to target the correct GPU operations, and added some new tests for the Host Memory allocations.