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

Jingsong Lee commented on BEAM-41:
----------------------------------

Yes, `add()` in CombiningState has no return value, so I can't override it 
directly. Now `addIfAbsent()` both write and read, as you say, not just a 
"blind write". MapState may has more such interfaces.
For Runners, it's a little harder to deal with SetState's readLater. Maybe we 
should provide a `readLater(Set<T>)` interface to tell runner?


> State for "Set" data, supporting efficient membership checks
> ------------------------------------------------------------
>
>                 Key: BEAM-41
>                 URL: https://issues.apache.org/jira/browse/BEAM-41
>             Project: Beam
>          Issue Type: New Feature
>          Components: beam-model
>            Reporter: Kenneth Knowles
>            Assignee: Jingsong Lee
>            Priority: Minor
>              Labels: State
>
> Analogous to how BagState supports efficient blind writes, a SetState would 
> support efficient membership checking without reading the entire set.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to