spmallette commented on code in PR #3234: URL: https://github.com/apache/tinkerpop/pull/3234#discussion_r2440263595
########## docs/src/upgrade/release-3.7.x.asciidoc: ########## @@ -30,6 +30,21 @@ complete list of all the modifications that are part of this release. === Upgrading for Users +==== Deprecation of Scopes in aggregate() Step + +Scopes are now deprecated inside the `aggregate()` step, and will be removed in the next major upgrade. Review Comment: I think it would be good to further include some tie in to `store` which is already deprecated: > This deprecation, in combination with the deprecation of `store` in x.x.x, will help users begin to consider the implications of their usage of these steps as it pertains to lazy versus eager flow control by converting to more explicit Gremlin patterns that have more consistent and intuitive semantics. That sentence can help lead into more details of those patterns and reasons "why" which @andreachild asked for. -- 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]
