spmallette opened a new pull request, #3569:
URL: https://github.com/apache/tinkerpop/pull/3569

   The prose in the PropertyMap section of 
`docs/src/reference/the-traversal.asciidoc` referred to the step as 
`propertiesMap()`, but the actual step is `propertyMap()` (no "s"). The 
section's executable examples already use the correct `propertyMap()`, so only 
the prose was wrong.
   
   This is worth correcting because the wrong name fails silently rather than 
loudly. In Groovy, `propertiesMap()` does not raise an error; it resolves to a 
lookup for a property key that does not exist and returns empty results. A 
newcomer copying the name from the prose therefore gets empty output with no 
error and no indication of what went wrong.
   
   The fix changes a single line of prose to `propertyMap()` so it matches the 
runnable examples directly below it. The examples remain the standard 
executable `[gremlin-groovy,modern]` blocks, so their output is rendered at 
build time (including the empty `==>[]` maps for vertices without the requested 
property, which illustrates the behavior clearly).


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to