Github user PBGraff commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/772#discussion_r161387751
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/traversal/step/map/PageRankVertexProgramStep.java
---
@@ -83,17 +83,17 @@ public String toString() {
}
@Override
- public PageRankVertexProgram generateProgram(final Graph graph, final
Memory memory) {
+ public PageRankVertexProgram generateProgram(final Memory memory,
final Graph... graphs) {
final Traversal.Admin<Vertex, Edge> detachedTraversal =
this.edgeTraversal.getPure();
-
detachedTraversal.setStrategies(TraversalStrategies.GlobalCache.getStrategies(graph.getClass()));
+
detachedTraversal.setStrategies(TraversalStrategies.GlobalCache.getStrategies(graphs[0].getClass()));
--- End diff --
Since it's called through a step, I think you're right that there should
only be one `Graph`. I will validate in the "steps" that use the `Graph`.
---
