-
Notifications
You must be signed in to change notification settings - Fork 791
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bytecode removal from Java GLV #2648
Conversation
Starting in 4.0, gremlin-driver only supports GraphBinary.
Updated to match old behavior where the incoming size of the data is checked so that it doesn't exceed the maxContentLength setting.
@@ -79,7 +79,7 @@ public GremlinScriptEngineFactory getFactory() { | |||
* Bytecode is evaluated by the {@link JavaTranslator}. | |||
*/ | |||
@Override | |||
public Traversal.Admin eval(final Bytecode bytecode, final Bindings bindings, final String traversalSource) throws ScriptException { | |||
public Traversal.Admin eval(final GremlinLang bytecode, final Bindings bindings, final String traversalSource) throws ScriptException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: rename variable as well to avoid confusion. I see this throughout all files, so prob need a search and update all references at some point.
public Traversal.Admin eval(final GremlinLang bytecode, final Bindings bindings, final String traversalSource) throws ScriptException { | |
public Traversal.Admin eval(final GremlinLang gremlinLang, final Bindings bindings, final String traversalSource) throws ScriptException { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope to remove this altogether in one of the next PRs, 'GremlinScriptEngineFactory' will handle only gremlin scripts.
@@ -177,7 +175,7 @@ | |||
import java.util.stream.Stream; | |||
|
|||
/** | |||
* Utility class for parsing {@link Bytecode}. | |||
* Utility class for parsing {@link GremlinLang}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: don't forget classname update
VOTE +1 |
VOTE +1 |
1 similar comment
VOTE +1 |
...n-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/BytecodeHelper.java
Show resolved
Hide resolved
VOTE +1 |
Java GLV will no longer send requests as Bytecode.
Feature tests
GraphBinaryRemoteFeatureTest
switched to run withgremlin-lang
due to minor incompatibilities withgremlin-groovy
.