optimize query

Author: jayushi-canyon

app-modules/alert/src/Presets/Handlers/NewInboundMessagePresetHandler.php

@@ -68,14 +68,14 @@ public function getConfigurationModel(): ?string
     public function getStudentAlertQuery(?AlertPresetConfiguration $configuration): Builder
     {
         return DB::table('students')
-            ->select('students.sisid')
-            ->join('engagement_responses', function (JoinClause $join) {
-                $join->on('students.sisid', '=', 'engagement_responses.sender_id')
-                    ->where('engagement_responses.sender_type', '=', 'student');
-            })
-            ->where('engagement_responses.status', EngagementResponseStatus::New->value)
-            ->whereNull('students.deleted_at')
-            ->whereNull('engagement_responses.deleted_at')
-            ->distinct();
+          ->select('students.sisid')
+          ->join('engagement_responses', function (JoinClause $join) {
+          $join->on('students.sisid', '=', 'engagement_responses.sender_id')
+          ->where('engagement_responses.sender_type', '=', 'student')
+          ->where('engagement_responses.status', EngagementResponseStatus::New->value)
+          ->whereNull('engagement_responses.deleted_at');
+          })
+          ->whereNull('students.deleted_at')
+          ->distinct();
     }
 }

app-modules/engagement/database/migrations/2026_01_21_102338_add_index_in_engagement_responses_table.php

@@ -42,14 +42,14 @@
     public function up(): void
     {
         Schema::table('engagement_responses', function (Blueprint $table) {
-            $table->index(['sender_type', 'sender_id', 'status']);
+            $table->index(['sender_id', 'sender_type','status']);
         });
     }
 
     public function down(): void
     {
         Schema::table('engagement_responses', function (Blueprint $table) {
-            $table->dropIndex(['sender_type', 'sender_id', 'status']);
+            $table->dropIndex(['sender_id', 'sender_type','status']);
         });
     }
 };