Launchpad itself

Code review comment for lp:~allenap/launchpad/show-me-too-counts

  1. show-me-too-counts
  2. Merge into devel
Revision history for this message
Gavin Panella (allenap) wrote :

Thanks Abel!

I noticed (using LP_DEBUG_SQL=1) that the many references to current_user_affected_status were causing queries to be issued to the db. I had assumed these would be cached, because they're against the primary key. Anyway, it's not critical, but I have a branch - lp:~allenap/launchpad/is-user-affected-performance - to address it, and I'll get that landed tomorrow.

« Back to merge proposal