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/05/27 04:01:10 UTC

[GitHub] [arrow] alexbaden commented on pull request #7263: ARROW-8927: [C++] Support dictionary memo in CUDA IPC ReadRecordBatch functions

alexbaden commented on pull request #7263:
URL: https://github.com/apache/arrow/pull/7263#issuecomment-633727647


   I think this is ready for a first look. The python side could probably some tests. I'm also interested in adding some functions which would pack both the schema, memo, and CUDA handle in IPC messages, similar to `ipc::RecordBatchStreamReader`, but that could probably be a separate PR. 
   
   I also didn't attempt backwards compatibility here. Let me know if that's desirable in these endpoints. It probably wouldn't be too hard to do, though the error you get if you don't have a dictionary memo is a little vague. 


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