pymongo count() function deprecated. Use count_documents instead

[wfehrnstrom]

https://api.mongodb.com/python/current/api/pymongo/collection.html#pymongo.collection.Collection.count_documents

[insiyab]

Just a note:
count_documents() is slower than count() as it's used in the codebase currently.
estimated_document_count() gives us the same behavior as count(), so I propose we use that instead.

The reason that count() was deprecated in the first place was because its behavior would be like the current count_documents() if it was given a query, and estimated_document_count() otherwise. In our codebase, we pretty much never give count() a query.
(https://stackoverflow.com/questions/52236594/why-is-pymongo-count-documents-is-slower-than-count)

[Kimeiga]

nice, test and pr when you have time *Hakan Alpay* Computer Science, UCLA

…

On Fri, May 22, 2020 at 4:10 PM Insiya Bambot ***@***.***> wrote: Just a note: count_documents() is slower than count() as it's used in the codebase currently. estimated_document_count() gives us the same behavior as count(), so I propose we use that instead. The reason that count() was deprecated in the first place was because its behavior would be like the current count_documents() if it was given a query, and estimated_document_count() otherwise. In our codebase, we pretty much never give count() a query. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#165 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AE5CFT45W7MV2SA4JQUNRYTRS4A5BANCNFSM4ME3XKTA> .