xiazcy commented on code in PR #2648:
URL: https://github.com/apache/tinkerpop/pull/2648#discussion_r1635171704
##########
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/jsr223/GremlinLangScriptEngine.java:
##########
@@ -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 {
Review Comment:
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.
```suggestion
public Traversal.Admin eval(final GremlinLang gremlinLang, final
Bindings bindings, final String traversalSource) throws ScriptException {
```
##########
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/util/BytecodeHelper.java:
##########
@@ -177,7 +175,7 @@
import java.util.stream.Stream;
/**
- * Utility class for parsing {@link Bytecode}.
+ * Utility class for parsing {@link GremlinLang}.
Review Comment:
nit: don't forget classname update
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@tinkerpop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
