-
Notifications
You must be signed in to change notification settings - Fork 252
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
local is unnecessary - Section 5.7 #105
Comments
The local step assures that you always work on the current result which is important as we are taking a single walk from one vertex to the next. I'll see if I can improve the wording. The TinkerPop reference docs address this a bit as well here: http://tinkerpop.apache.org/docs/current/reference/#sample-step Note that in your modified query you no longer inspect the 'dist' property. |
I have been thinking about this quite a bit more. I think some additional examples where local is not used, with explanations would be a good addition. I am going to think about this some more before I make any edits. Thanks for the issue. |
The TinkerPop official documentation is getting an update. I will update the book to reflect these clarifications. |
Fixed in Release 280 |
The text suggests
local
is needed for the random walk, but this query without it (and a spuriousby
step) appears to work also:I don't understand what bearing
local
has on this query. Iflocal
is removed from the example, some of the explanation before this example also needs attention.The text was updated successfully, but these errors were encountered: