xiazcy opened a new pull request, #3233: URL: https://github.com/apache/tinkerpop/pull/3233
As follow-up to the Lazy vs Eager evaluation [proposal](https://github.com/apache/tinkerpop/blob/master/docs/src/dev/future/proposal-scoping-5.asciidoc), removed `Scope` from `aggregate('x')`, which removes `AggregateLocalStep`, as well as the deprecated equivalent step `store()`. `AggregateGlobalStep` is now `AggregateStep` with default globally scoped (eager aggregation). To achieve lazy aggregation, use the existing step `local()`, which is designed for flow control. Updated relevant docs and added additional tests to ensure `local(aggregate('x'))` is semantically equivalent to `(aggregate(local, 'x'))`. <!-- Thanks for contributing! Reminders: + TARGET the earliest branch where you want the change 3.7-dev -> 3.7.5 (non-breaking only) 3.8-dev -> 3.8.0 master -> 4.0.0 + Committers will MERGE the PR forward to newer versions + ADD entry to the CHANGELOG.asciidoc for the targeted version Do not reference a JIRA number there + ADD JIRA number to title and link in description + PRs requires 3 +1s from committers OR 1 +1 and 7 day wait to merge. + MORE details: https://s.apache.org/rtnal --> -- 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]
