You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "marin-ma (via GitHub)" <gi...@apache.org> on 2023/02/03 10:22:44 UTC

[GitHub] [arrow] marin-ma opened a new issue, #34025: [C++] Arrow IPC buffer compression forces the use of default_memory_pool

marin-ma opened a new issue, #34025:
URL: https://github.com/apache/arrow/issues/34025

   ### Describe the bug, including details regarding any error messages, version, and platform.
   
   In the CompressBuffer, it calls AllocateBuffer without passing the memory_pool argument https://github.com/apache/arrow/blob/177a0d4400e61f44dbd045ce05150bba28dd1d74/cpp/src/arrow/ipc/writer.cc#L183
   
   Is there any special reason why it doesn't use the memory_pool in ipc write options? In our use case, we want to manage the memory allocation and avoid using the default memory pool.
   
   ### Component(s)
   
   C++


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] pitrou closed issue #34025: [C++] Arrow IPC buffer compression forces the use of default_memory_pool

Posted by "pitrou (via GitHub)" <gi...@apache.org>.

pitrou closed issue #34025: [C++] Arrow IPC buffer compression forces the use of default_memory_pool
URL: https://github.com/apache/arrow/issues/34025


-- 
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: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow] wjones127 commented on issue #34025: [C++] Arrow IPC buffer compression forces the use of default_memory_pool

Posted by "wjones127 (via GitHub)" <gi...@apache.org>.

wjones127 commented on issue #34025:
URL: https://github.com/apache/arrow/issues/34025#issuecomment-1416174949

   > Is there any special reason why it doesn't use the memory_pool in ipc write options?
   
   This is a bug. There are some places where we may have forgotten to pass down the memory pool. I'm working right now on a PR that will allow us to assert in tests that all allocations happen on the provided memory pool.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org