Merge pull request #161 from zorun/optimise_sql

Optimise sql queries

budget/models.py

@@ -37,7 +37,7 @@ def balance(self):
         # for each person
         for person in self.members:
             # get the list of bills he has to pay
-            bills = Bill.query.filter(Bill.owers.contains(person))
+            bills = Bill.query.options(orm.subqueryload(Bill.owers)).filter(Bill.owers.contains(person))
             for bill in bills.all():
                 if person != bill.payer:
                     share = bill.pay_each() * person.weight

budget/web.py

@@ -15,6 +15,7 @@
 from flask.ext.babel import get_locale, gettext as _
 from smtplib import SMTPRecipientsRefused
 import werkzeug
+from sqlalchemy import orm
 
 # local modules
 from models import db, Project, Person, Bill
@@ -277,7 +278,8 @@ def list_bills():
     # set the last selected payer as default choice if exists
     if 'last_selected_payer' in session:
         bill_form.payer.data = session['last_selected_payer']
-    bills = g.project.get_bills()
+    # Preload the "owers" relationship for all bills
+    bills = g.project.get_bills().options(orm.subqueryload(Bill.owers))
 
     return render_template("list_bills.html",
             bills=bills, member_form=MemberForm(g.project),