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

stephen mallette updated TINKERPOP-1122:
----------------------------------------
    Fix Version/s:     (was: 3.3.1)

> StarGraph has a Long nextId. That is pointless and a waste of 64-bits.
> ----------------------------------------------------------------------
>
>                 Key: TINKERPOP-1122
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-1122
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: hadoop, io, structure
>    Affects Versions: 3.1.1-incubating
>            Reporter: Marko A. Rodriguez
>              Labels: breaking
>
> {code}
> protected Long nextId = 0l;
>     private Long nextId() {
>         return this.nextId++;
>     }
> {code}
> This makes no sense as then each StarGraph in the full Graph will have 
> similar ids! Moreover, what is referencing what when the adjacent vertices 
> are just arbitrary long ids?!!  We should require that StarGraph get provided 
> ids for vertices (and perhaps edges)... We ensure no inconsistencies and we 
> save 64-bits.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to