spmallette opened a new pull request, #3572:
URL: https://github.com/apache/tinkerpop/pull/3572
The two `group()`-step examples in `reference/the-traversal.asciidoc` that
project `age` first as a key and then as a value were annotated with symmetric
callouts, both saying the non-productive entries are "filtered". Their runtime
behavior is actually asymmetric:
- A non-productive **key**-projection (`by('age').by('name')`) drops the
vertex from the result map entirely — no key is produced for vertices lacking
an `age` value.
- A non-productive **value**-projection (`by('name').by('age')`) keeps the
key (since `name` is productive for every vertex) but leaves its value list
empty, e.g. `ripple=[]`, `lop=[]`.
The old symmetric wording made the `{ripple:[], lop:[]}` output surprising.
This change rewrites callout `<2>` to explain that keys are retained with empty
value lists, and updates callout `<1>` to state the entry is dropped entirely,
drawing an explicit contrast between the two cases. The `productive`
cross-reference is preserved.
Only the two callouts changed; the example code lines are untouched and
still render their live output at build time, which confirms the reworded prose
matches what the reader sees.
--
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]
