1
82%
develop: 84%

Ran 23 Feb 2015 11:21PM UTC

Files 288

Run time 16s

Badge

Embed ▾

Committed 23 Feb 2015 11:09PM UTC coverage: 81.638%. First build

Job # 315.1

Build Type

push

travis-ci

Committed by

jwalgran

Commit Message

Update user audit query to be more efficient

Trying to view user profiles other than your own with a large database
was triggering a 500 response. The root cause was the fact that the ORM
was generating a 2.2MB SQL statement, causing Postgres to choke with a
stack error.

The audit query code in lib/user.py builds up the final QuerySet by
gluing together several small parts. I discovered that the same instance
was being fed into the filter multiple times, causing the ORM to
generate a lot of duplicate clauses, bloating the query. Adding a
``distinct`` took care of that.

The fast majority of users will only edit a single instance, and our
audit table is optimized for filtering by instance ID, so I added an
additional query to fetch the ids of a users edited instances and use
that to limit the scope of the query, rather than limiting it to the
larger list of visible instances.

Run Details

13929 of 17062 relevant lines covered (81.64%)

0.82 hits per line

OpenTreeMap / otm-core / 315 / 1
82%
develop: 84%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 315.1

OpenTreeMap / otm-core / 315 / 1 82% develop: 84%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 315.1

OpenTreeMap / otm-core / 315 / 1
82%
develop: 84%

README BADGES
x