- **status**: open --> wont-fix
- **Comment**:

These have not been performance issues, and using skip() allows us to keep 
track of page & limit params quite easily, whereas the more effecient query by 
_id greater than X, requires keeping track of what page goes with what _id 
limit, etc.



---

** [tickets:#5043] Replace inefficient uses of skip() with ranged queries**

**Status:** wont-fix
**Milestone:** forge-backlog
**Created:** Wed Oct 03, 2012 03:32 PM UTC by Cory Johns
**Last Updated:** Wed Oct 03, 2012 03:32 PM UTC
**Owner:** nobody

The following files have uses of `skip()` which the Mongo documentation 
[indicates](http://www.mongodb.org/display/DOCS/Advanced+Queries#AdvancedQueries-%7B%7Bskip%28%29%7D%7D)
 is inefficient and should be replaced with ranged-based paging whenever 
possible.

* `Allura/allura/controllers/discuss.py`
* `Allura/allura/controllers/project.py`
* `Allura/allura/controllers/search.py`
* `Allura/allura/controllers/site_admin.py`
* `Allura/allura/ext/admin/admin_main.py`
* `Allura/allura/model/discuss.py`
* `ForgeBlog/forgeblog/main.py`
* `ForgeDiscussion/forgediscussion/controllers/forum.py`
* `ForgeShortUrl/forgeshorturl/main.py`
* `ForgeTracker/forgetracker/model/ticket.py`
* `ForgeWiki/forgewiki/wiki_main.py`

When possible, in the above files, replace the use of `skip()` with a 
combination of sorting on, and range limiting by an indexed field.


---

Sent from sourceforge.net because [email protected] is subscribed to 
https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a 
mailing list, you can unsubscribe from the mailing list.

Reply via email to