[ 
https://issues.apache.org/jira/browse/TINKERPOP-1404?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15529178#comment-15529178
 ] 

stephen mallette commented on TINKERPOP-1404:
---------------------------------------------

The PR is essentially the way to trigger review. If you don't want a VOTE for 
some reason and just want review of diffs from certain folks, you can say so in 
the PR description.

> Path/label optimization
> -----------------------
>
>                 Key: TINKERPOP-1404
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1404
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.2.1
>            Reporter: pieter martin
>            Assignee: pieter martin
>              Labels: performance
>
> Currently path queries do a lot of label collection copying. This has a 
> significant impact on performance.
> As the labels are known and set on the traverser when {{Traverser.split(r, 
> step)}} is called there is no need to call {{Traverser.addLabels}} again in 
> {{AbstractStep}}
> Also seeing as {{AbstractStep.getLabels()}} returns an {{UnmodifyableSet}} 
> the step's labels can be used directly in the traverser. There is no need to 
> make a copy of it.



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

Reply via email to