GitHub user xenji opened a pull request:
https://github.com/apache/incubator-tinkerpop/pull/250
Make the GraphManager instance injectable into ServerGremlinExecutor based
on tp31
This refers to PR #246 and bases the changes on the branch tp31.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/xenji/incubator-tinkerpop
make-graphmanager-injectable-tp31
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-tinkerpop/pull/250.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #250
----
commit 164b88ae1c139a2541c28b21795836318ed82246
Author: Mario Mueller <[email protected]>
Date: 2016-02-29T20:16:14Z
Make the GraphManager instance injectable into ServerGremlinExecutor
This refers to a discussion from the gremlin-users mailing list:
https://groups.google.com/forum/#!topic/gremlin-users/xYzCAQExvvM
It is currently not possible to use a standard way to inject a
fully configured graph. The way presented in this gist is only
a hack and should not be takes as the default way.
https://gist.github.com/xenji/e469abde2e0b80aadc4e
Adding a new constructor parameter is, from my point of view,
the most flexible and compatible change to enable the injection
without breaking old code.
I've run the whole test suite of the server before pushing without
failures.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
