You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Siyuan Zhuang (JIRA)" <ji...@apache.org> on 2019/02/18 04:42:00 UTC

[jira] [Comment Edited] (ARROW-4418) [Plasma] replace event loop with boost::asio for plasma store

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

Siyuan Zhuang edited comment on ARROW-4418 at 2/18/19 4:41 AM:
---------------------------------------------------------------

I will try to create a PR soon. The only problem is the standalone asio does not include boost::bind, which might be used in our implementation (We have already used boost::bind in a similar case in Ray project, and the official asio examples also use it). I will try std::bind first if it is inevitable.


was (Author: suquark):
I will try to create a PR soon. The only problem is the standalone asio does not include boost::bind, which might be used in our implementation (We have already used boost::bind in a similar case in Ray project, and the official asio examples also use it).

> [Plasma] replace event loop with boost::asio for plasma store
> -------------------------------------------------------------
>
>                 Key: ARROW-4418
>                 URL: https://issues.apache.org/jira/browse/ARROW-4418
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++ - Plasma
>            Reporter: Zhijun Fu
>            Assignee: Siyuan Zhuang
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 50m
>  Remaining Estimate: 0h
>
> Original text:
> It would be nice to move plasma store from current event loop to boost::asio to modernize the code, and more importantly to benefit from the functionalities provided by asio, which I think also provides opportunities for performance improvement.



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