User tests: Successful: Unsuccessful:
There are situations where the index of Smart Search can get poluted with old data (unused terms and taxonomies, already deleted content items) and where the only solution so far has been to clear the reindex and start indexing all over again. This is not something that you really want to do in a live scenario with a large site, so this attempts some different solutions.
First of all, it runs a garbage collection, which by default deletes all entries, which aren't present in the base table of the base component. We then delete all orphaned term and taxonomy mappings as well as all then orphaned taxonomies and terms. In the end, we then execute the already previously present optimisation queries. To trigger all of this, a new dropdown button was added in the backend named "Maintenance", where you can choose between purging the index and doing the optimisation.
Note: The garbage collection has been added as a default to the Adapter base class and there could be situations where this results in a fail. I'm open for ideas here.
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_finder Language & Strings Repository NPM Change JavaScript |
Labels |
Added:
Language Change
NPM Resource Changed
?
|
This task is not something that you should execute regularly.
Labels |
Added:
?
Removed: ? |
I have tested this item
After applieing the patch I get an error in seachrindex - indes:
There is no "table.columns" asset of a "script" type in the registry.
Can you please check if you need to do a composer install and/or npm i? This PR actually does not change anything in that regard, but maybe a previous change introduced a problem here?
I have tested this item
I've tested it with the prebuilt package and sample data. Searching was working before and after click on "Optimise".
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
The last change was a minor change, so I restored test result of @webnet-assmann. This PR is now RTC.
Labels |
Added:
?
|
I'm investigating why the api tests are failing. Please don't merge this right now. I want to check that first.
I found the issue and fixed it. This could be merged now.
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-05-27 13:32:33 |
Closed_By | ⇒ | fancyFranci |
Thank you Hannes!
Would it be possible to add that task to the new com_scheduler ?
Removing that responsibility from the admin would be great.