rmm
rmm copied to clipboard
[FEA] Remove factory functions from resource adaptors
Consider removing factory functions with constructors in prefetch_resource_adaptor, statistics_resource_adaptor, logging_resource_adaptor and others in rmm.
More context at:
question: can't this just be a constructor? > Why do we need a factory function?
Originally posted by @jrhemstad in https://github.com/rapidsai/rmm/pull/1608#discussion_r1674886164
### Tasks
- [x] Change calls to adaptor factories across RAPIDS repos to use constructors
- [ ] https://github.com/rapidsai/rmm/issues/1641
- [ ] Remove deprecated adaptor factories in 24.12
All of these classes already have constructors. I think what you mean is we should remove the factory functions.
I opened two PRs, #1625 and #1626, that should help with this.
Added tasklist to this issue.