Code review comment for lp:~adiroiban/launchpad/bug-525371

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

For the record I should add that we first discussed the bigger issues (performance, security, various batching hazards) as per Danilo's proposed course of action above. All those points have been satisfied, so the nitpicking about small bits of code means that the branch is in good shape. There is no problematic snapshotting, performance is good, and with the removal of that shortlist we no longer fetch all templates in a distroseries at once.

« Back to merge proposal