-
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
* made client cconn code consistent * added admonition * fixed link * moved note into INCLUDES and literal included * updated to rst * made node external * Attempt to fix formatting * included note in usage ex * removed unneeded comma * moved useUnifiedTopology out of uri * updated location of note * placement of connection-guide link node * Removed unneeded spacing
DOCSP-9585 builders sort criteria Co-authored-by: Chris Cho <[email protected]>
* DOCSP-42308: Deployment and connection string (#52) * DOCSP-42308: Deployment and connection string * add file * DOCSP-42306: Get Started and Install (#51) * Get Started and Install * build * edits * wording * troubleshooting file * version * JS feedback * DOCSP-42310: Connect to MongoDB (#53) * DOCSP-42310: Connect to MongoDB * edits * code highlight * reduce scroll * DOCSP-42305: Index page + repo cleanup (#56) * DOCSP-42321: Insert landing page (#55) * DOCSP-42329: Retrieve data (#54) * DOCSP-42329: Retrieve data * edits * MM feedback * DOCSP-42321: Insert Documents (#61) * DOCSP-42328: Specify a query (#57) * DOCSP-42328: Specify a query * link fix * edit * JS feedback * output * RL feedback * DOCSP-42330: Projection guide (#58) * DOCSP-42330: Projection guide * edits * add constant * MM fix * DOCSP-42323: Replace (#62) * DOCSP-42331: Specify docs to return (#64) * DOCSP-42331: Specify docs to return * toc * MM feedback * code output * DOCSP-42322: Update (#63) * DOCSP-42332: Count documents (#66) * DOCSP-42332: Count documents * edits * build errors * RR feedback * scroll * build error * DOCSP-42312 Create a MongoDB Client (#59) * DOCSP-42312 Create a MongoDB Client * on TOC * retry TOC * build * api doc * fix links * code fix * review comments * test * test * fix snooty * merge error * DOCSP-42324: Delete (#65) * DOCSP-42325: Bulk Write (#68) * DOCSP-42317: Databases and Collections (#77) * DOCSP-42333: Distinct values (#67) * DOCSP-42333: Distinct values * edits * build error * RR feedback * DOCSP-42335: Change streams (#69) * DOCSP-42335: Change streams * code * edits * fixes * toc * MW feedback * MW feedback * fix * DOCSP-42311: Run a Command (#75) * DOCSP-42311: Run a Command * edits * code edits * MM feedback * JY feedback * DOCSP-42318: Time Series (#78) * DOCSP-42326: GridFS (#72) * DOCSP-42342: Aggregation (#71) * DOCSP-42342: Aggregation * edits * code imports * RR feedback * import fixes * JY feedback * DOCSP-42316: Stable API (#81) * DOCSP-42336: Cluster monitoring (#73) * DOCSP-42336: Cluster monitoring * edits * fixes, code edit * MM feedback * remove unused imports * fix indent * DOCSP-42341: Atlas Search Indexes (#82) * Uncomment link to compat tables on landing page (#84) * DOCSP-40287: Read landing page (#85) * DOCSP-42343: Secure Your Data + Authentication Mechanisms (#83) * DOCSP-42319: Read and write settings (#79) * DOCSP-42319: Read and write settings * edits * code comments * MW feedback * code edits * JY feedback * imports * edits * JY feedback 2 * build * DOCSP-42338 Single Field Indexes (#70) * DOCSP-42338 Single Field Indexes * new content * first draft * fix toc * try section heading * fixes * fix links * ref fix * review comments * add helper method import * helper method explanation * edit helpers section * edicts * edit * edit out helpers * edict copy * fix database * unnecessary imports * DOCSP-42339 Compound Indexes (#74) * DOCSP-42339 Compound Indexes * toc * fix examples * 1 review comment * api comment * spacing * remove helpers * fix scroll * edits * edit output spacing * DOCSP-42314: Enable TLS (#88) * DOCSP-42314: Enable TLS * edits * code comments * intro edit * DOCSP-42337: Optimize Queries with Indexes (#92) * DOCSP-42349: Transactions (#89) * DOCSP-42349: Transactions * edits * JS feedback * JY feedback * DOCSP-42340 Multikey Index (#76) * DOCSP-42340 Multikey Index * fixes * multikey index link * text fix * internal review comments * remover helpers * edits * uri * DOCSP-42313 Connection Targets (#80) * DOCSP-42313 Connection Targets * toc * fixes * code fix * small change * sode file endline spacing * review edits * comment out connection option page * Correct connect section TOC * DOCSP-42334: Observables (#86) * DOCSP-42334: Observables * edits * fix error * MM feedback * edits * word * MM feedback 2 * edit * build * JY feedback * JY feedback 2 * edit * DOCSP-24494: Atlas search section (#90) * DOCSP-24494: Atlas search section * edits * imports * admonitions * code output * rename headers * MR feedback * edits * JY feedback * DOCSP-42346: Upgrade driver (#91) * DOCSP-42346: Upgrade driver * edits * MW feedback * edits, link fixes * MW final feedback * JY feedback * DOCSP-45354: TOC labels (#94) * add labels * LM feedback * DOCSP-45348: Cleanup (#93) * DOCSP-45348: Cleanup * edits * TODOs * add back in-use encryption * edits * sharedinclude fixes * fix link * RR feedback * staging link * fixes --------- Co-authored-by: Michael Morisi <[email protected]> Co-authored-by: lindseymoore <[email protected]> Co-authored-by: Maya Raman <[email protected]>
# Pull Request Info Feature branch to aggregate usage examples before merging into `master`. [PR Reviewing Guidelines](https://github.com/mongodb/docs-java/blob/master/REVIEWING.md) JIRA - N/A Staging - https://docs-mongodbcom-staging.corp.mongodb.com/kotlin/docsworker-xlarge/usage_examples/ ## Self-Review Checklist - [ ] Is this free of any warnings or errors in the RST? - [ ] Did you run a spell-check? - [ ] Did you run a grammar-check? - [ ] Are all the links working? --------- Co-authored-by: cbullinger <[email protected]> Co-authored-by: Rea Rustagi <[email protected]>
* DOCSP-46446: v5.3 release * code language fix
* added CRUD Delete page
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.