Uploaded image for project: 'TinkerPop'
  1. TinkerPop
  2. TINKERPOP-1122

StarGraph has a Long nextId. That is pointless and a waste of 64-bits.

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: 3.1.1-incubating
    • Fix Version/s: None
    • Component/s: hadoop, io, structure
    • Labels:

      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.

        Attachments

        Issue Links

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              okram Marko A. Rodriguez

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment