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

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?