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/06/18 22:46:49 UTC

[GitHub] [arrow] wesm opened a new pull request #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

wesm opened a new pull request #7487:
URL: https://github.com/apache/arrow/pull/7487


   This also splits apart table_test.cc into multiple compilation units. I did some cleanup of includes but more can certainly be done going forward


----------------------------------------------------------------
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 #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

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


   ah thank you


----------------------------------------------------------------
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 #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

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


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


----------------------------------------------------------------
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 #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

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


   I'll rebase and fix the test failures, but absent objections I'd like to merge this before it causes more conflicts


----------------------------------------------------------------
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] nealrichardson commented on pull request #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

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


   FYI @wesm I just pushed to this branch


----------------------------------------------------------------
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 closed pull request #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

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


   


----------------------------------------------------------------
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 #7487: ARROW-8800: [C++] Split ChunkedArray into arrow/chunked_array.h/cc

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


   +1


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