Marko A. Rodriguez created TINKERPOP-1606:
---------------------------------------------
Summary: Refactor GroupStep to not have the reduction traversal
included in its BiOperator.
Key: TINKERPOP-1606
URL: https://issues.apache.org/jira/browse/TINKERPOP-1606
Project: TinkerPop
Issue Type: Improvement
Components: process
Affects Versions: 3.2.3
Reporter: Marko A. Rodriguez
{{GroupStep}} has a complicated {{BiOperator}} (its reducer). I believe we can
simplify it significantly by considering only two states.
1. A {{by()}}-modulation that does NOT have a {{Barrier}}.
2. A {{by()}}-modulation that DOES have a {{Barrier}}.
For the first, simply store a single {{Traverser}}. No need to aggregate all
traversers as only one will ultimately be emitted.
For the latter, simply store the barrier's BiOperator (NOT the step itself).
In this way, we will no longer have to encode a traversal in the reducer and
thus, will remove various potential problems associated with detached
traversals.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
