The proposal to merge lp:~rvb/launchpad/builders-timeout-bug-887078-eager-load into lp:launchpad has been updated.
Description changed to: The ultimate goal is to make the number of queries issued by builder:+history independent of the number of builds. We have 3 types of builds displayed on these pages: BinaryPackageBuild, SourcePackageRecipeBuild and TranslationTemplatesBuild. This branch focuses on the elements of type BinaryPackageBuild and SourcePackageRecipeBuild. It eager loads data when the build farm jobs are fetched by getByBuildFarmJobs (BinaryPackageBuildSet and SourcePackageRecipeBuild). - BinaryPackageBuild: the number of queries is now independent of the number of BinaryPackageBuilds \o/. - SourcePackageRecipeBuild: I've improved things seriously here but each new SourcePackageRecipeBuild displayed generates 2 additional queries: this is because I've no idea how to fetch what SourcePackageRecipe._recipe_data fetches in bulk so that it's properly cached by Storm. Also note that the test test_build_history_queries_count_view_recipe_builds does not tests the entire page generation but only the data preparation that is done by the view class. If we manage to cache SourcePackageRecipe._recipe_data somehow then more work could be done to make the whole page rendering issue a constant number of queries. = Tests = ./bin/test -vvc test_builder_views test_build_history_queries_count_view_recipe_builds ./bin/test -vvc test_builder_views test_build_history_queries_count_binary_package_builds = Q/A = None. For more details, see: https://code.launchpad.net/~rvb/launchpad/builders-timeout-bug-887078-eager-load/+merge/82008 -- https://code.launchpad.net/~rvb/launchpad/builders-timeout-bug-887078-eager-load/+merge/82008 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~rvb/launchpad/builders-timeout-bug-887078-eager-load into lp:launchpad. _______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
