[ARROW-2551] [Plasma] Improve notification logic

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.10.0
• Component/s: C++ - Plasma
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/15712
• Assignee: Zhijun Fu
• Reporter: Zhijun Fu
• Created: 08/May/18 08:12
• Updated: 11/Jan/23 07:21
• Resolved: 09/Jun/18 01:02

Description

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.

Issue Links

• links to: GitHub Pull Request #2028
• links to: GitHub Pull Request #2031