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/21 14:40:47 UTC
[GitHub] [arrow] pitrou opened a new pull request #8498: ARROW-10207: [C++] Allow precomputing output string/list offsets in kernels
pitrou opened a new pull request #8498:
URL: https://github.com/apache/arrow/pull/8498
----------------------------------------------------------------
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] bkietz closed pull request #8498: ARROW-10207: [C++] Allow precomputing output string/list offsets in kernels
Posted by GitBox <gi...@apache.org>.
bkietz closed pull request #8498:
URL: https://github.com/apache/arrow/pull/8498
----------------------------------------------------------------
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 #8498: ARROW-10207: [C++] Allow precomputing output string/list offsets in kernels
Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #8498:
URL: https://github.com/apache/arrow/pull/8498#issuecomment-713627106
The main complication here is with the "contiguous preallocation" feature. I'm not sure how much of it is desirable, since if you have a chunked input, you may want it to remain chunked...
----------------------------------------------------------------
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] wesm commented on pull request #8498: ARROW-10207: [C++] Allow precomputing output string/list offsets in kernels
Posted by GitBox <gi...@apache.org>.
wesm commented on pull request #8498:
URL: https://github.com/apache/arrow/pull/8498#issuecomment-713892264
Hm, I am not sure whether the contiguous allocation feature is worth the trouble. I'll review this
----------------------------------------------------------------
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 #8498: ARROW-10207: [C++] Allow precomputing output string/list offsets in kernels
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #8498:
URL: https://github.com/apache/arrow/pull/8498#issuecomment-713632620
https://issues.apache.org/jira/browse/ARROW-10207
----------------------------------------------------------------
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 #8498: ARROW-10207: [C++] Allow precomputing output string/list offsets in kernels
Posted by GitBox <gi...@apache.org>.
pitrou commented on pull request #8498:
URL: https://github.com/apache/arrow/pull/8498#issuecomment-714629869
Rebased and fixed conflict.
----------------------------------------------------------------
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