This repository was archived by the owner on Nov 1, 2024. It is now read-only.
Commit 2e06b53
committed
Use rootMemoryPool to construct execContext (#480)
Summary:
Pull Request resolved: #480
It originally use `memory::getDefaultScopedMemoryPool()` since `ExecCtx`
constructor expects `unique_ptr<MemoryPool>`.
Since facebookincubator/velox@15eeddb,
`ExecCtx` now expects `MemoryPool*`, so we can use
`&velox::memory::getProcessDefaultMemoryManager().getRoot()` returned by
`TorchArrowGlobalStatic::rootMemoryPool`.
This makes TorchArrow always use this process default root memory pool.
Reviewed By: dracifer, vancexu
Differential Revision: D39010989
fbshipit-source-id: 3a383b01163e9866500a6fc8b091080b70684c6e1 parent 1e996c5 commit 2e06b53
1 file changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
553 | 553 | | |
554 | 554 | | |
555 | 555 | | |
556 | | - | |
557 | 556 | | |
558 | | - | |
| 557 | + | |
| 558 | + | |
559 | 559 | | |
560 | 560 | | |
561 | 561 | | |
| |||
0 commit comments