Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update to use the Gremlin 2.0 API #2

Merged
merged 1 commit into from May 29, 2012

Conversation

Projects
None yet
3 participants
Contributor

varju commented May 28, 2012

Most of the changes here are just removing the ".pgm" references from package names.

The one change of note is the E and V methods in GremlinScalaPipeline. I'm not keen on the manualStart method I added to support these, but couldn't think of a more graceful way of accomplishing this since Scala doesn't allow multiple constructors to call different super constructors.

Contributor

varju commented May 28, 2012

Ha. Just noticed the other pull request for this same feature enhancement. mpollmeier's work looks more thorough than mine, so there's probably little of value to grab from my version. We can probably close this.

okram added a commit that referenced this pull request May 29, 2012

Merge pull request #2 from varju/gremlin-2.0
Update to use the Gremlin 2.0 API

@okram okram merged commit c124aa9 into tinkerpop:master May 29, 2012

@varju looks like your pr won the race here for whatever reason ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment