xiazcy commented on code in PR #3243: URL: https://github.com/apache/tinkerpop/pull/3243#discussion_r2441364942
########## docs/src/upgrade/release-3.8.x.asciidoc: ########## @@ -305,6 +305,32 @@ The `asString()` step will no longer allow `null` input. An `IllegalArgumentExce See: link:https://lists.apache.org/thread/q76pgrvhprosb4lty63bnsnbw2ljyl7m[DISCUSS] thread +==== Removal of has(key, traversal) + +The current `has(key, traversal)` implementation has long caused confusion, as it only checks if the traversal produces +any result, rather than performing equality comparison with the traversal's output value. While fixing this behavior is +desirable, it would require extensive changes. Therefore, we are removing the current `has(key, traversal)` API as a +breaking change in version 3.8.0, with plans to reintroduce a properly implemented version in the next major release. + +Users needing to filter with traversals should use the `where()` step instead, which provides the expected traversal-based Review Comment: Makes sense, where() is just a convenient option, I'll make the focus on the confusion instead of dictating the one alternative. -- 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]
