[
https://issues.apache.org/jira/browse/TINKERPOP-1606?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Marko A. Rodriguez closed TINKERPOP-1606.
-----------------------------------------
Resolution: Fixed
Fix Version/s: 3.2.4
> 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
> Assignee: Marko A. Rodriguez
> Fix For: 3.2.4
>
>
> {{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)
