Github user spmallette commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/772#discussion_r160840348
--- Diff:
gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/process/computer/ranking/pagerank/PageRankVertexProgram.java
---
@@ -75,7 +75,11 @@ private PageRankVertexProgram() {
}
@Override
- public void loadState(final Graph graph, final Configuration
configuration) {
+ public void loadState(final Configuration configuration, final
Graph... graphs) {
+ if (graphs.length != 1) {
+ throw new IllegalStateException("Must provide one graph to
use, received " + graphs.length);
+ }
+ Graph graph = graphs[0];
--- End diff --
Please add a `final` here as `graph` isn't re-assigned.
---
