Optimization of Reindexing in Laravel Scout about scout

[jjdevzinho]

Another idea could be to leverage the toSearchableArray method that already exists in Scout. This method returns an array of the model's attributes that should be indexed.

Instead of creating a new getSearchableFields method, Scout could check if any of the keys returned by toSearchableArray were changed before reindexing the model. This would avoid the need to define the searchable fields in two places and make the reindexing process more efficient.

Here's an example of how this could work:

// Get the keys from the toSearchableArray method
$searchableFields = array_keys($model->toSearchableArray());

// Check if any of these fields were changed
foreach ($searchableFields as $field) {
    if ($model->isDirty($field)) {
        // If a searchable field was changed, reindex the model
        $model->searchable();
        break;
    }
}

This approach would require changes to the Scout core, but it could be a more efficient way to control when reindexing occurs.

Thank you for considering my suggestion and this additional idea.

[driesvints]

We'd very much appreciate PR's which could help with optimisation. For these PR's please also include the proper benchmarks. Thanks