orientechnologies / orientdb-gremlin

TinkerPop3 Graph Structure Implementation for OrientDB
Apache License 2.0
91 stars 32 forks source link

OrientGraph.getIndexedVertices() what it's good for? #53

Open velo opened 8 years ago

velo commented 8 years ago

Hey @mpollmeier,

What is this getIndexedVertices() do we have any use case for it?

https://coveralls.io/builds/4645245/source?filename=driver%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Ftinkerpop%2Fgremlin%2Forientdb%2FOrientGraph.java#L210

mpollmeier commented 8 years ago

it's used by OrientGraphStep.vertices() - that's a graph traversal strategy that optimises the vertex lookup by trying to find an index. without this, it would always skim through all vertices. I would have suspected that OrientGraphIndexTest does use it...

velo commented 8 years ago

Apparently it does not....

otherwise the coverage wouldn't be zero...