Github user okram commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/112#issuecomment-149407781
Bulk and sack do not play well together when you are dealing with "energy
flows." Please see the existing 3.0.1-SNAPSHOT docs for a simple example:
```
gremlin>
g.withSack(1.0f,sum).V(1).local(outE('knows').barrier(normSack).inV()).in('knows').barrier().sack()
//(5)
==>1.0
==>1.0
gremlin>
g.withSack(1.0f,sum).V(1).local(outE('knows').barrier(normSack).inV()).in('knows').barrier().sideEffect{it.setBulk(1)}.sack()
//(6)
==>1.0
```
```
5. The barrier() merges the two traversers at vertex 1 into a single
traverser whose sack is 1.0.
6. There is now a single traverser with bulk of 2 and sack of 1.0 and thus,
setting the bulk to 1 yields the expected 1.0.
```
Now look at what changes I made to the docs and see how you no longer need
`sideEffect{it.setBulk(1)}`.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
