You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "Ovidiu Marcu (JIRA)" <ji...@apache.org> on 2018/05/17 10:25:00 UTC
[jira] [Comment Edited] (ARROW-2551) [Plasma] Improve notification
logic
[ https://issues.apache.org/jira/browse/ARROW-2551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16478859#comment-16478859 ]
Ovidiu Marcu edited comment on ARROW-2551 at 5/17/18 10:24 AM:
---------------------------------------------------------------
Thanks for #2028, it fixed a bug when the client crashed the second run will crash the plasma store (could change issue type from improvement to bug?).
I am using plasma notifications and I have some issues:
1) I do not get objects in order of their creation and
2) it seems some objects are somehow lost.
was (Author: ovidiumarcu):
Thanks for #2028, it fixed a bug when the client crashed the run will crash the plasma store (could change issue type from improvement to bug?).
I am using plasma notifications and I have some issues:
1) I do not get objects in order of their creation and
2) it seems some objects are somehow lost.
> [Plasma] Improve notification logic
> -----------------------------------
>
> Key: ARROW-2551
> URL: https://issues.apache.org/jira/browse/ARROW-2551
> Project: Apache Arrow
> Issue Type: Improvement
> Components: Plasma (C++)
> Reporter: Zhijun Fu
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 1.5h
> Remaining Estimate: 0h
>
> There are a few issues in current Plasma notification code:
> * When a client subscribes to Plasma, the store pushes notifications about existing objects to ALL subscribers, while it should only push to the new subscriber.
> * And in the above scenario, it should only push "sealed" objects to the new subscriber, while currently it pushes all objects regardless of the state.
> * When a client disconnects, it will no longer be able to receive notifications, thus the NotificationQueue for the client should be removed from global map.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)