[
https://issues.apache.org/jira/browse/TINKERPOP-1256?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
stephen mallette closed TINKERPOP-1256.
---------------------------------------
Resolution: Won't Do
Looks like an attempt was made at this one, but it did not turn into any real
success. "rabbit hole" sounds foreboding to future efforts - closing.
> Remove Traverser.split(R r, Step<T,R> step)
> -------------------------------------------
>
> Key: TINKERPOP-1256
> URL: https://issues.apache.org/jira/browse/TINKERPOP-1256
> Project: TinkerPop
> Issue Type: Improvement
> Components: process
> Affects Versions: 3.2.0-incubating
> Reporter: Marko A. Rodriguez
> Priority: Major
>
> I just noticed that {{Traverser.split(R r, Step<T,R> step)}} has no
> implementation that actually uses the {{step}} argument. This is an appendage
> from back in the day when it was always guaranteed that the step would set
> labels and stepId. This is not the case with many OLAP variations of a step.
> I think we should do the following.
> 1. Figure out how to get rid of {{privat e
> AbstractStep.prepareTraversalForNextStep()}} as this is an blemish.
> 2. Create {{Traverser.split(R r)}}.
> 3. Then labels are added via
> {{traverser.setPath(traverser.getPath().extend(labels))}}.
> Not a big ticket item, just some nice backwards compatible house-cleaning to
> remove technical debt.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
