search

heynemann / motorengine

Motorengine is a port of MongoEngine for Tornado.
http://motorengine.readthedocs.org
204 stars 64 forks source link

Adds a default limit for find_all when not specified #62

Closed btaitelb closed 10 years ago

btaitelb commented 10 years ago

Starting with https://github.com/eguven/motor/commit/89f9085a05, limit is now a required positional argument for find_all in motor, which is why so many tests are failing on travis.

This is a potentially breaking change, so I'd like some feedback on if this is the right approach (and if so, what a good default would be).

heynemann commented 10 years ago

That's good enough. Thanks for the pull request.