Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove child fom newCudaAsyncMemoryResource (#12681)
Fixes a bug introduced here #12632 where the C++ version of the jni call `Rmm.newCudaAsyncMemoryResource` was taking an extra argument (`child`), causing the `threshold` to be set to a random value since the calling code defined the native method has taking 2 longs, not 3. The above can affect performance if the threshold is set to 0 for example. The async pool will reduce its footprint by releasing memory, which means we need to go back and reallocate. This is how I found it. I am starting the build process for this, so it's going to take me a bit to test. Authors: - Alessandro Bellina (https://github.com/abellina) Approvers: - Jason Lowe (https://github.com/jlowe) URL: #12681
- Loading branch information