[jira] [Closed] (TINKERPOP-872) Remove GroupCountStep in favor of new Reduce-based GroupStep

["Marko A. Rodriguez (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/TINKERPOP-872?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Marko A. Rodriguez closed TINKERPOP-872.
----------------------------------------
    Resolution: Won't Fix

Will not do. Because we know we only counting, the optimizations that {{groupCount()}} has over {{group()}} as tremendous.

> Remove GroupCountStep in favor of new Reduce-based GroupStep
> ------------------------------------------------------------
>
>                 Key: TINKERPOP-872
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-872
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: process
>    Affects Versions: 3.0.2-incubating
>            Reporter: Marko A. Rodriguez
>            Assignee: Marko A. Rodriguez
>             Fix For: 3.2.0-incubating
>
>         Attachments: group.svg, groupCount.svg
>
>
> Check it. In the {{group_step_2/}} branch, we can now do this efficiently as we now support "on the fly" reduction.
> {code}
> gremlin> g.V().both().both().group().by('name').by(count())
> ==>[ripple:3, peter:3, vadas:3, josh:7, lop:7, marko:7]
> gremlin> g.V().both().both().groupCount().by('name')
> ==>[ripple:3, peter:3, vadas:3, josh:7, lop:7, marko:7]
> {code}
> Now, stand back... 
> {code}
> gremlin> clock(10000){g.V().both().both().groupCount().by('name').next()}
> ==>0.0415874797
> gremlin> clock(10000){g.V().both().both().group().by('name').by(count()).next()}
> ==>0.08681611759999999
> {code}
> Speed differences are reasonable and I know various places in the new {{GroupStep}} that I can optimize to make more efficient.
> And that is how the dishes get done.



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