Code review comment for lp:~jcsackett/launchpad/bugs-in-deactivated-pillars

Revision history for this message
j.c.sackett (jcsackett) wrote :

> In https://bugs.launchpad.net/launchpad/+bug/421901/comments/14 we had a more
> complex join - we enforce the active flag in the join, which reduces the data
> brought into intermediary tables. Was there a particular reason for eliding
> that aspect here?

As I mentioned on IRC, the only reason was distraction in assembling the python code. The proper join has been pushed up.

« Back to merge proposal