[ 
https://issues.apache.org/jira/browse/ARROW-1394?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16352044#comment-16352044
 ] 

ASF GitHub Bot commented on ARROW-1394:
---------------------------------------

robertnishihara commented on a change in pull request #1445: ARROW-1394: 
[Plasma] Add optional extension for allocating memory on GPUs
URL: https://github.com/apache/arrow/pull/1445#discussion_r165881777
 
 

 ##########
 File path: cpp/src/plasma/client.cc
 ##########
 @@ -75,7 +85,24 @@ struct ObjectInUseEntry {
   bool is_sealed;
 };
 
-PlasmaClient::PlasmaClient() {}
+#ifdef PLASMA_GPU
+struct GpuProcessHandle {
+  std::shared_ptr<CudaBuffer> ptr;
+  int client_count;
+};
 
 Review comment:
   document these fields and the struct

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [Plasma] Add optional extension for allocating memory on GPUs
> -------------------------------------------------------------
>
>                 Key: ARROW-1394
>                 URL: https://issues.apache.org/jira/browse/ARROW-1394
>             Project: Apache Arrow
>          Issue Type: New Feature
>          Components: Plasma (C++)
>            Reporter: Wes McKinney
>            Priority: Major
>              Labels: pull-request-available
>
> It would be useful to be able to allocate memory to be shared between 
> processes via Plasma using the CUDA IPC API



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to