[ 
https://issues.apache.org/jira/browse/TINKERPOP-1607?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

stephen mallette updated TINKERPOP-1607:
----------------------------------------
    Component/s: process

> withSack() should not be a requirement for the use of sack()
> ------------------------------------------------------------
>
>                 Key: TINKERPOP-1607
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1607
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.2.3
>            Reporter: Daniel Kuppitz
>
> {{withSack()}} initializes the sack value, but also adds the 
> {{SackStrategy}}. Since we can do {{sack(assign)}} anywhere in the traversal, 
> it's not obvious that you have to initialize the sack value upfront. However, 
> if you don't do it, you end up getting NPEs (I think it's because 
> {{SackStrategy}} was not added).
> In short: {{SackStrategy}} should not only be added by {{withSack()}}, but 
> also whenever you add a {{SackStep}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to