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

Support $sum accumulator of $group aggregation #2292

Merged
merged 14 commits into from
Mar 29, 2023
Merged

Support $sum accumulator of $group aggregation #2292

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
6a0f709
init
chilagrow Mar 28, 2023
36cc33e
more impl
chilagrow Mar 28, 2023
54fe096
comment and refactor
chilagrow Mar 28, 2023
776fa6a
comment
chilagrow Mar 28, 2023
93e7b33
lint
chilagrow Mar 28, 2023
1970142
use remove instead of list providers
chilagrow Mar 28, 2023
09faa1a
update
chilagrow Mar 28, 2023
dd74f8a
refactor
chilagrow Mar 28, 2023
45e8620
update comment
chilagrow Mar 28, 2023
68267bc
tests are run parallel
chilagrow Mar 28, 2023
45524e8
rename
chilagrow Mar 28, 2023
445c001
resolve merge conflict
chilagrow Mar 29, 2023
4074a01
Merge branch 'main' into sum-aggregation-operator
AlekSi Mar 29, 2023
cb29b5b
Merge branch 'main' into sum-aggregation-operator
mergify[bot] Mar 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
lint
  • Loading branch information
@chilagrow
chilagrow committed Mar 28, 2023
commit 93e7b339153122bdf673da4d031271f9900450d0
4 changes: 3 additions & 1 deletion integration/aggregate_compat_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -932,7 +932,9 @@ func TestAggregateCompatGroupSum(t *testing.T) {
{"_id", "$v"},
{"sum", bson.D{{"$sum", "$v"}}},
{"sum", bson.D{{"$sum", "$s"}}},
}}}},
}}},
bson.D{{"$sort", bson.D{{"_id", -1}}}},
},
resultType: emptyResult,
},
}
Expand Down