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.

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

Sum does not support summing of aggregates #149

Closed
Tracked by #28
AndrewSisley opened this issue Jan 27, 2022 · 0 comments · Fixed by #341
Closed
Tracked by #28

Sum does not support summing of aggregates #149

AndrewSisley opened this issue Jan 27, 2022 · 0 comments · Fixed by #341
Assignees
Labels
area/query Related to the query component feature New feature or request
Milestone

Comments

@AndrewSisley
Copy link
Contributor

AndrewSisley commented Jan 27, 2022

Child of #28

Is a bit of a pain that I forgot about this, as it prevents the surfacing of inner aggregates to the top of query.

For example (test and very partial implementation in branch sisley/feat/I149-sum-of-sums):

                                query {
					users(groupBy: [Name]) {
						Name
						_sum(field: {_group: _sum})
						_group (groupBy: [Verified]){
							Verified
							_sum(field: {_group: Age})
						}
					}
				}
@AndrewSisley AndrewSisley added feature New feature or request area/query Related to the query component labels Jan 27, 2022
@AndrewSisley AndrewSisley self-assigned this Jan 27, 2022
@AndrewSisley AndrewSisley added this to the DefraDB v0.2.1 milestone Jan 27, 2022
@AndrewSisley AndrewSisley changed the title feat: Sum does not support summing of aggregates Sum does not support summing of aggregates Jan 28, 2022
@jsimnz jsimnz modified the milestones: DefraDB v0.2.1, DefraDB v0.3 Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/query Related to the query component feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants