Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Document that store is deprecated in favor of aggregate(local) #187

Open
krlawrence opened this issue Jun 8, 2020 · 0 comments
Open

Document that store is deprecated in favor of aggregate(local) #187

krlawrence opened this issue Jun 8, 2020 · 0 comments
Assignees
@krlawrence
Labels
consistency fix-in-second-edition New Gremlin features

Comments

@krlawrence
Copy link
Owner

krlawrence commented Jun 8, 2020 •
edited
Loading

Starting with TinkerPop release 3.4.3.

Added aggregate(Scope,String) and deprecated store() in favor of aggregate(local).

A note should be added in the section "3.22. Using aggregate to create a temporary collection" and in the prior section where store is introduced the role of store should be better clarified

Given that aggregate(local) feels unwieldy and also confuses users, it is unlikely that store will ever be fully removed from the language.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@krlawrence krlawrence

Labels
consistency fix-in-second-edition New Gremlin features
Projects
Planning
Status: Low priority
Milestone
No milestone
Development

No branches or pull requests
1 participant
@krlawrence