spmallette opened a new pull request, #3533:
URL: https://github.com/apache/tinkerpop/pull/3533
The Order Step section of the reference documentation
(`docs/src/reference/the-traversal.asciidoc`) claimed that when the property
key being sorted on does not exist on an element, that element is treated as
`null` and placed first for `Order.asc` or last for `Order.desc`.
This is incorrect. Elements missing the sort key are silently filtered out
of the result, not ranked at the ends. Because the statement is false rather
than merely imprecise, this PR removes it (and the example that demonstrated
the claimed behavior) rather than rewording it.
This PR removes three items from the Order Step section:
1. The false sentence: "If a property key does not exist, then it will be
treated as `null` which will sort it first for `Order.asc` and last for
`Order.desc`."
2. The example that demonstrated the claimed null-sort behavior:
`g.V().order().by("age").elementMap()`.
3. The obsolete `NOTE: Prior to version 3.3.4, ...` callout about the
long-removed `Order.incr`/`Order.decr` tokens.
The rest of the Order Step section is unchanged; it now flows directly from
the `Column` values/keys NOTE to the Additional References.
--
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]
