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

Prevent property(id,null) and addV(null)

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Done
    • 3.5.1
    • 3.6.0
    • process

    Description

      for g.addV().property(id,null) isn't quite the same as the String key version because the id can't be null and it can't behave like a removal for graphs that don't support null values. allow this might also create hard to figure out bugs where a variable for the id gets set to null accidentally and random data gets created rather than getting an error. There is a similar argument to be made for addV(null).

      at this time, there is a test that enforces this behavior so to remove that changes behavior which means we probably need to save this for a breaking change.

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            spmallette Stephen Mallette
            spmallette Stephen Mallette
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment