save-buffer commented on code in PR #13669:
URL: https://github.com/apache/arrow/pull/13669#discussion_r956385050
##########
cpp/src/arrow/memory_pool.h:
##########
@@ -71,21 +73,25 @@ class ARROW_EXPORT MemoryPool {
/// Allocate a new memory region of at least size bytes.
///
/// The allocated region shall be 64-byte aligned.
- virtual Status Allocate(int64_t size, uint8_t** out) = 0;
+ Status Allocate(int64_t size, uint8_t** out) { return Allocate(size,
kDefaultAlignment, out); }
Review Comment:
No I'm pretty sure I added it to the other allocators. There are four that I
could find: Jemalloc, system allocator, mimalloc allocator, and stl_allocator.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
