[TINKERPOP-1122] StarGraph has a Long nextId. That is pointless and a waste of 64-bits.

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 3.1.1-incubating
• Component/s: hadoop, io, structure
• Labels: breaking
• Assignee: Unassigned
• Reporter: Marko A. Rodriguez
• Created: 03/Feb/16 22:34
• Updated: 01/Mar/18 21:26
• Resolved: 01/Mar/18 21:26

Description

protected Long nextId = 0l;

    private Long nextId() {
        return this.nextId++;
    }

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.

Issue Links

• is related to: TINKERPOP-1343 A more efficient StarGraph serialization representation.
• relates to: TINKERPOP-1128 Change the Gryo serialization for StarGraph (Vertex, Properties, then Edges)