[
https://issues.apache.org/jira/browse/ARROW-2435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16432012#comment-16432012
]
ASF GitHub Bot commented on ARROW-2435:
---------------------------------------
liurenjie1024 commented on a change in pull request #1875: ARROW-2435: [Rust]
Add memory pool abstraction.
URL: https://github.com/apache/arrow/pull/1875#discussion_r180362225
##########
File path: rust/src/memory_pool.rs
##########
@@ -0,0 +1,54 @@
+use libc;
+use std::sync::Arc;
+use std::mem;
+
+use super::error::ArrowError;
+use super::error::Result;
+
+const ALIGNMENT: usize = 64;
+
+pub trait MemoryPool {
+ fn allocate_aligned(&self, size: usize) -> Result<*mut u8>;
Review comment:
Hi, xhochy:
Thanks for the review and I've added an implementation for that method.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> [Rust] Add memory pool abstraction.
> -----------------------------------
>
> Key: ARROW-2435
> URL: https://issues.apache.org/jira/browse/ARROW-2435
> Project: Apache Arrow
> Issue Type: Improvement
> Components: Rust
> Affects Versions: 0.9.0
> Reporter: Renjie Liu
> Assignee: Renjie Liu
> Priority: Major
> Labels: pull-request-available
>
> Add memory pool abstraction as the c++ api.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
