When there is no data a import, scout:reimport will not replace the destination index

[borealsmith]

Hello,

I have a model for which, on purpose I define the shouldBeSearchable method so that to always returns false. Despite this, the model is imported in my index when I run php artisan scout:reimport.

Please note that for the same model I overwritten the toSearchableArray method.

Thank you.

[borealsmith]

Note that If I flush the index and thatn I run scout:import it works as expected; i.e only records for which shouldBeSearchable() returns true are imprted in the index.

[nunomaduro]

@borealsmith Could you check if that issue also exists on the scout:import?

[borealsmith]

@nunomaduro As I mentioned above scout:import seems to work correctly

[lkagan]

Hopefully I can provide more info on this. I'm experiencing a similar issue.

When returning false for shouldBeSearchable() for all models, I expect a scout:status to show 0 remote records but it shows the same number that was previously displayed.

An API call is successfully POSTed to /1/indexes/temp_<index-name>/operation with an operation of move and a destination of <index-name>.

I would guess that the server side service does not like empty indices and refuses to copy the temp over to the production index.

I hope this helps!

[nunomaduro]

@lkagan Exacly. Thanks for finding the issue.

Any idea for the solution? I would say: If the temporary index is empty, just clear the production index. What do you think?

[nunomaduro]

@borealsmith Do you agree with the problem discussed here?

[lkagan]

@nunomaduro Clearing the production index sounds like a viable solution if the temp index is empty. Although it feels a bit dirty. If a move command is issued to the API, one would expect that command to be run, regardless of index size.

[borealsmith]

@lkagan Thank you for the further infos / clarification.
@nunomaduro for the solution I would say than in a way (clear) or another (move) if reimport results in an empty tmp index this should be reflected in production as the tmp index represents the latest state of the app models.

[nunomaduro]

@borealsmith Do you wanna work on this? 👍

[borealsmith]

@nunomaduro Yes, I would be glad to work on this :)

[borealsmith]

What do you think about the following ?
It the reimport results in an empty index than prompt a question to the user asking if he wants to replace the production index with an empty index ?
Thank you.

[nunomaduro]

I would say: Just clear the production index. No need for asking.

[borealsmith]

@nunomaduro Thank you for your answer. I am fine with it.