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

ASF GitHub Bot commented on TINKERPOP-1870:
-------------------------------------------

Github user artem-aliev commented on a diff in the pull request:

    https://github.com/apache/tinkerpop/pull/778#discussion_r162065650
  
    --- Diff: 
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/io/gryo/GryoVersion.java
 ---
    @@ -305,6 +307,7 @@ public String getVersion() {
                 add(GryoTypeReg.of(RangeGlobalStep.RangeBiOperator.class, 
114));
                 add(GryoTypeReg.of(OrderGlobalStep.OrderBiOperator.class, 118, 
new JavaSerializer()));
                 add(GryoTypeReg.of(ProfileStep.ProfileBiOperator.class, 119));
    +            add(GryoTypeReg.of(VertexTraverserSet.class, 173));
    --- End diff --
    
    master PR: https://github.com/apache/tinkerpop/pull/781
    I have make both tp32 and tp33 ids the same. so I skipped 171,172 in  tp32


> n^2 synchronious operation in OLAP WorkerExecutor.execute() method
> ------------------------------------------------------------------
>
>                 Key: TINKERPOP-1870
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1870
>             Project: TinkerPop
>          Issue Type: Improvement
>    Affects Versions: 3.2.7, 3.3.1
>            Reporter: Artem Aliev
>            Priority: Major
>         Attachments: findTraverser1.png, findTraverser2.png, 
> findTraverserFixed.png
>
>
> [https://github.com/apache/tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/WorkerExecutor.java#L80-L93]
> This block of code iterates over all remote traverses to select one related 
> to the current vertex and remove it. This operation is repeated for the next 
> vertex and so one. For following example query it means n^2 operations (n is 
> number of vertices). All of them in sync block. multi core spark executor 
> will do this operations serial. 
> {code}
> g.V().emit().repeat(both().dedup()).count().next()
> {code}
> See jvisualvm screenshot. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to