[
https://issues.apache.org/jira/browse/ARROW-8180?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17063852#comment-17063852
]
Antoine Pitrou commented on ARROW-8180:
---------------------------------------
If you don't put it in type_fwd.h then every function declaration with a
{{MemoryPool* pool = default_memory_pool()}} needs to include
{{memory_pool.h}}, no?
> [C++] Should default_memory_pool() be in arrow/type_fwd.h?
> ----------------------------------------------------------
>
> Key: ARROW-8180
> URL: https://issues.apache.org/jira/browse/ARROW-8180
> Project: Apache Arrow
> Issue Type: Improvement
> Components: C++
> Reporter: Wes McKinney
> Priority: Major
>
> This seemed somewhat odd to me. It might be better from an IWYU-perspective
> to move this to arrow/memory_pool.h
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
