perf: Add another index to surveyitemresponses. Implement bullet suggestions. Memoize a function in measure.rb

app/controllers/overview_controller.rb

@@ -25,10 +25,10 @@ class OverviewController < SqmApplicationController
   end
 
   def measures
-    @measures ||= Measure.all.includes(%i[scales admin_data_items category])
+    @measures ||= Measure.all.includes(%i[scales admin_data_items category subcategory])
   end
 
   def subcategories
-    @subcategories ||= Subcategory.all
+    @subcategories ||= Subcategory.all.includes(:measures)
   end
 end

app/models/measure.rb

@@ -25,6 +25,7 @@ class Measure < ActiveRecord::Base
   end
 
   def student_survey_items_with_sufficient_responses(school:, academic_year:)
+    @student_survey_items_with_sufficient_responses ||=
       SurveyItem.where(id: SurveyItem.joins('inner join survey_item_responses on survey_item_responses.survey_item_id = survey_items.id')
                                       .student_survey_items
                                       .where("survey_item_responses.school": school,

db/migrate/20230610165508_add_survey_item_index_to_survey_item_responses.rb

@@ -0,0 +1,6 @@
+class AddSurveyItemIndexToSurveyItemResponses < ActiveRecord::Migration[7.0]
+  def change
+    add_index :survey_item_responses, %i[school_id academic_year_id survey_item_id],
+              name: 'by_school_year_and_survey_item'
+  end
+end

db/schema.rb

@@ -10,8 +10,9 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2023_04_21_034505) do
+ActiveRecord::Schema[7.0].define(version: 2023_06_10_165508) do
   # These are extensions that must be enabled in order to support this database
+  enable_extension "pg_stat_statements"
   enable_extension "plpgsql"
 
   create_table "academic_years", id: :serial, force: :cascade do |t|
@@ -359,7 +360,6 @@ ActiveRecord::Schema[7.0].define(version: 2023_04_21_034505) do
     t.datetime "updated_at", null: false
     t.index ["academic_year_id"], name: "index_response_rates_on_academic_year_id"
     t.index ["school_id", "subcategory_id"], name: "index_response_rates_on_school_id_and_subcategory_id"
-    t.index ["school_id"], name: "index_response_rates_on_school_id"
     t.index ["subcategory_id"], name: "index_response_rates_on_subcategory_id"
   end
 
@@ -447,9 +447,9 @@ ActiveRecord::Schema[7.0].define(version: 2023_04_21_034505) do
     t.index ["academic_year_id"], name: "index_survey_item_responses_on_academic_year_id"
     t.index ["gender_id"], name: "index_survey_item_responses_on_gender_id"
     t.index ["response_id"], name: "index_survey_item_responses_on_response_id"
+    t.index ["school_id", "academic_year_id", "survey_item_id"], name: "by_school_year_and_survey_item"
     t.index ["school_id", "academic_year_id"], name: "index_survey_item_responses_on_school_id_and_academic_year_id"
     t.index ["school_id", "survey_item_id", "academic_year_id", "grade"], name: "index_survey_responses_on_grade"
-    t.index ["school_id"], name: "index_survey_item_responses_on_school_id"
     t.index ["student_id"], name: "index_survey_item_responses_on_student_id"
     t.index ["survey_item_id"], name: "index_survey_item_responses_on_survey_item_id"
   end