From cbc5329b3d1439f61b5fd561b70a3edfcc8094d5 Mon Sep 17 00:00:00 2001 From: Konstantin Haase Date: Fri, 21 Aug 2015 18:26:21 +0200 Subject: [PATCH] v3: avoid N+1 query in builds endpoint --- lib/travis/api/v3/queries/builds.rb | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/travis/api/v3/queries/builds.rb b/lib/travis/api/v3/queries/builds.rb index 0877bac5..319ee9bd 100644 --- a/lib/travis/api/v3/queries/builds.rb +++ b/lib/travis/api/v3/queries/builds.rb @@ -1,7 +1,12 @@ module Travis::API::V3 class Queries::Builds < Query def find(repository) - repository.builds + filter(repository.builds) + end + + def filter(list) + # filtering by branch, type, etc would go here + list.includes(:commit).includes(branch: :last_build) end end end