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 2020/10/22 15:52:58 UTC
[GitHub] [arrow] pitrou opened a new pull request #8511: ARROW-7531: [C++] Reduce header inclusion cost slightly
pitrou opened a new pull request #8511:
URL: https://github.com/apache/arrow/pull/8511
Remove inclusion of heavyweights "arrow/api.h" and "arrow/builder.h", among other changes.
This seems to reduce CPU build time by about 3%.
----------------------------------------------------------------
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 closed pull request #8511: ARROW-7531: [C++] Reduce header inclusion cost slightly
Posted by GitBox <gi...@apache.org>.
pitrou closed pull request #8511:
URL: https://github.com/apache/arrow/pull/8511
----------------------------------------------------------------
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 #8511: ARROW-7531: [C++] Reduce header inclusion cost slightly
Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #8511:
URL: https://github.com/apache/arrow/pull/8511#issuecomment-724115565
Rebased, will merge if CI is green.
----------------------------------------------------------------
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 #8511: ARROW-7531: [C++] Reduce header inclusion cost slightly
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #8511:
URL: https://github.com/apache/arrow/pull/8511#issuecomment-714589640
https://issues.apache.org/jira/browse/ARROW-7531
----------------------------------------------------------------
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 #8511: ARROW-7531: [C++] Reduce header inclusion cost slightly
Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #8511:
URL: https://github.com/apache/arrow/pull/8511#issuecomment-714588829
I'll note the main offender when compiling the tests is `gtest.h`.
----------------------------------------------------------------
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