315
82%
develop: 84%

Ran 23 Feb 2015 11:21PM UTC

Jobs 1

Files 288

Run time 18s

Badge

Embed ▾

pending completion

Build # 315

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

Jobs

ID	Job ID	Ran	Files	Coverage
1	315.1	23 Feb 2015 11:21PM UTC	0	81.64	Travis Job 315.1

Source Files on build 315

Detailed source file information is not available for this build.

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 315

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 315

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

README BADGES
x