Code review comment for lp:~rvb/launchpad/builders-timeout-bug-887078-eager-load

Thanks for the review.

> I end up doing this every time. I'll put a branch in ec2 that calls
> flush_database_updates() in StormStatementRecorder.__enter__().

Good idea ;)

> [2]
>
> + build_farm_jobs.extend([build.build_farm_job for build in builds2])
>
> No need for the intermediate list, though I guess it does little harm.

Okay.

> [3]
>
> + # rvb 2011-11-11: Each build issues 2 new queries.
> + # This is because of the way SourcePackageRecipe._recipe_data
> + # fetches SourcePackageRecipeData. I've no idea how to prefetch
> + # this.
>
> I guess this is work in progress, but should probably be XXX'd
> anyway. What do you think?

I'll keep it as is but I'll work on a branch to fix this right away.