You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/04/07 15:35:08 UTC

[GitHub] [arrow] pitrou opened a new pull request #9930: ARROW-12248: [C++] Avoid looking up ARROW_DEFAULT_MEMORY_POOL environment variable too late

pitrou opened a new pull request #9930:
URL: https://github.com/apache/arrow/pull/9930


   In some situations (e.g. R bindings), default_memory_pool() may be called before the Arrow library's global variables were fully initialized.


-- 
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.

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



[GitHub] [arrow] pitrou commented on pull request #9930: ARROW-12248: [C++] Avoid looking up ARROW_DEFAULT_MEMORY_POOL environment variable too late

Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #9930:
URL: https://github.com/apache/arrow/pull/9930#issuecomment-815013010


   @jonkeane 


-- 
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.

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



[GitHub] [arrow] lidavidm closed pull request #9930: ARROW-12248: [C++] Avoid looking up ARROW_DEFAULT_MEMORY_POOL environment variable too late

Posted by GitBox <gi...@apache.org>.
lidavidm closed pull request #9930:
URL: https://github.com/apache/arrow/pull/9930


   


-- 
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.

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



[GitHub] [arrow] jonkeane edited a comment on pull request #9930: ARROW-12248: [C++] Avoid looking up ARROW_DEFAULT_MEMORY_POOL environment variable too late

Posted by GitBox <gi...@apache.org>.
jonkeane edited a comment on pull request #9930:
URL: https://github.com/apache/arrow/pull/9930#issuecomment-815091253


   This looks good to me, I tested it locally and in at least the situation I was building (static r package build) I can now change allocators. Thanks!


-- 
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.

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



[GitHub] [arrow] jonkeane commented on pull request #9930: ARROW-12248: [C++] Avoid looking up ARROW_DEFAULT_MEMORY_POOL environment variable too late

Posted by GitBox <gi...@apache.org>.
jonkeane commented on pull request #9930:
URL: https://github.com/apache/arrow/pull/9930#issuecomment-815091253


   This looks good to me, I tested it locally and in at least the situation I was building (static r package build) I can now change allocators. Thanks@


-- 
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.

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



[GitHub] [arrow] github-actions[bot] commented on pull request #9930: ARROW-12248: [C++] Avoid looking up ARROW_DEFAULT_MEMORY_POOL environment variable too late

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #9930:
URL: https://github.com/apache/arrow/pull/9930#issuecomment-815237592


   https://issues.apache.org/jira/browse/ARROW-12248


-- 
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.

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