[tinkerpop] branch master updated: Added a note to explain why sparql-gremlin needs to be on the server for remote g.sparql() requests CTR

[sp...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

spmallette pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tinkerpop.git


The following commit(s) were added to refs/heads/master by this push:
     new 9a6976c  Added a note to explain why sparql-gremlin needs to be on the server for remote g.sparql() requests CTR
9a6976c is described below

commit 9a6976cc867d8e8d7aeb7fe5149e9fdf55a5faef
Author: Stephen Mallette <sp...@genoprime.com>
AuthorDate: Fri Apr 5 14:29:18 2019 -0400

    Added a note to explain why sparql-gremlin needs to be on the server for remote g.sparql() requests CTR
---
 .../gremlin/sparql/process/traversal/strategy/SparqlStrategy.java     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/sparql-gremlin/src/main/java/org/apache/tinkerpop/gremlin/sparql/process/traversal/strategy/SparqlStrategy.java b/sparql-gremlin/src/main/java/org/apache/tinkerpop/gremlin/sparql/process/traversal/strategy/SparqlStrategy.java
index 99b64af..65a042c 100644
--- a/sparql-gremlin/src/main/java/org/apache/tinkerpop/gremlin/sparql/process/traversal/strategy/SparqlStrategy.java
+++ b/sparql-gremlin/src/main/java/org/apache/tinkerpop/gremlin/sparql/process/traversal/strategy/SparqlStrategy.java
@@ -38,6 +38,10 @@ import java.util.Set;
  * it as a string value. This strategy finds that step and compiles it to a Gremlin traversal which then replaces
  * the {@link InjectStep}.
  *
+ * For remote bytecode execution, note that the {@code sparql-gremlin} dependencies need to be on the server. The
+ * way remoting works - see {@code RemoteStep} - prevents modified bytecode from being submitted to the server, so
+ * technically, this strategy can't be applied first on the client.
+ *
  * @author Stephen Mallette (http://stephen.genoprime.com)
  */
 public class SparqlStrategy extends AbstractTraversalStrategy<TraversalStrategy.DecorationStrategy>