- **labels**: performance --> performance, 42cc
- **summary**: Upgrade to ming 0.4 to avoid extraneous count() queries --> 
Upgrade to ming 0.4.x to avoid extraneous count() queries
- Description has changed:

Diff:

~~~~

--- old
+++ new
@@ -1,5 +1,3 @@
 Ming 0.4 removes the `__len__` method which does a count() query.  This is so 
that list(query) and query.all() don't cause that to run any more.
 
 We'll need to find & change all our `len(query)` and `query|length` (in 
templates) to `query.count()`.
-
-If we're not comfortable with finding these manually via test runs, searching, 
and exercising all parts of Allura, we could potentially add support in Ming to 
inspect the frame within `__len__` and if it's not a call from `list()` then 
raise a DeprecationWarning.  (We can't simply raise DeprecationWarning all the 
time since list(query) automatically calls `__len__` under the covers)

~~~~




---

** [tickets:#6431] Upgrade to ming 0.4.x to avoid extraneous count() queries**

**Status:** open
**Labels:** performance 42cc 
**Created:** Mon Jul 01, 2013 02:53 PM UTC by Dave Brondsema
**Last Updated:** Mon Jul 01, 2013 02:53 PM UTC
**Owner:** nobody

Ming 0.4 removes the `__len__` method which does a count() query.  This is so 
that list(query) and query.all() don't cause that to run any more.

We'll need to find & change all our `len(query)` and `query|length` (in 
templates) to `query.count()`.


---

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