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

HasContainer should allow a null key

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Done
    • 3.5.2
    • 3.6.0, 3.5.3
    • process
    • None

    Description

      TINKERPOP-2605 changed null semantics to allow null as an argument in more places so that has(null) should not throw a NPE any more and instead filter all traversers out.

      HasContainer however still throws a NPE if the key is null.

      I discovered this when trying to update JanusGraph as that folds in a HasContainer like this which leads to the NPE and thus failing tests.

      The same behavior can also be produced with just TinkerGraph and for example the following traversals:

      g.V().has(null, 1)
      g.V().has(null, P.neq(null))

      Attachments

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: