-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Added an example for an aggregate key as this was not clear. #59
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
Conversation
I'm going to talk to some of the folks working on the aggregation stuff, to get their thoughts on this change. If you could fill out the contributor agreement, http://www.10gen.com/contributor, that would expedite the merge process. Thanks! |
OK, done. By the way, I over-complicated the example. Where I listed
you could just use
Ho hum. |
Sorry for the long turn around I've finally merged this (and made the correction as you recommended), and if you run into other issues or problems like this, I look forward to seeing additional pull requests! Cheers, |
* DOCS-15427 Adds System Collection limitation * DOCS-15427 Adds System Collection limitation * DOCS-15427 Fixes per Joe * DOCS-15427 Fixes per Joe
* DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration page * DOCSP-22863 Configuration Setting/Option Map * DOCSP-22863 Configuration Setting/Option Map * DOCSP-22863 Configuration introduction * DOCSP-22863 Test cluster0 include * DOCSP-22863 Include updates * DOCSP-22863 Spelling * DOCSP-22863 Typo * DOCSP-22863 header content * DOCSP-22863 Alphabetizes CLI options * DOCSP-22863 Reorders TOC * DOCSP-22863 Fixes per Alan Zheng * DOCSP-22863 Fixes per Ian Fogelman * DOCSP-22863 Fixes per Ian Fogelman * DOCSP-22863 Fixes per Alan Zheng * DOCSP-22863 Fixes build error * DOCSP-22863 Fixes build error * DOCSP-22863 Fixes build error * DOCSP-22863 Fixes build error * DOCSP-23350 write-blocking note (#52) * DOCSP-23350 write-blokcing note * staging updates * Review feedback * Review feedback * Review feedback * Review feedback * Review feedback * DOCSP-23337 users and roles not synched (#55) * DOCSP-23337 users and roles not synched * Review feedback * Review feedback * DOCS-15427 System Collection Limitations (#59) * DOCS-15427 Adds System Collection limitation * DOCS-15427 Adds System Collection limitation * DOCS-15427 Fixes per Joe * DOCS-15427 Fixes per Joe * Merge with upstream master * Merge with upstream master * Merge with upstream master * Merge with upstream master * DOCSP-22863 Alphabetizes CLI options * DOCSP-22863 Build errors * DOCSP-22863 Build errors * DOCSP-22863 Build errors * DOCSP-22863 Fixes per Tim Fogarty * DOCSP-22863 Increases depth on mongosync * DOCSP-22863 Fixes per Tim Fogarty * DOCSP-22863 Fixes build error Co-authored-by: Dave Cuthbert <[email protected]>
[mongosh] sharding methods
…59) * DOCSP-28311 Update VSCode Collection and Database Creation Procedure * * * * * * * * * IR Feedback * remove comma * Update source/databases-collections.txt Co-authored-by: Rhys <[email protected]> * XR --------- Co-authored-by: Rhys <[email protected]>
DOCSP-33867 1.3.2 Release Notes
DOCSP2589: MongoDB Instance page revisions
…#59) * (DOCSP-24159) Adds note that users with >500 orgs must filter results * Includes change from Corry's copy review
(cherry picked from commit 722dfcc1c8557180bb617f24c801a168da83b3d3)
* DOCSP-26355-startup-mode-properties * first pass fixes * add old refs back * VK technical review 1 * fixes * RL PR fixes 1 * small fixes * MW PR fixes 1
Grouping over multiple fields is a clear use case for many people but this is not shown in any examples here and the format required for the aggregate key is inconsistent (you do not specify $name but simply name) so adding an example would probably save a few people some frustration, esp. those not coming from a background of using the previous collection.group method.