Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of the Change
Refactors the multi site handling so that when blogs are updated (marked as spam, archived or deleted), their indexes are also removed from the ES server. Also protects global indexes, like User, from being deleted.
Alternate Designs
Benefits
Keeps backend servers clean of unwanted data
Possible Drawbacks
It's possible for data meant to be kept to remove. Users should use
ep_keep_index
to keep the data.Verification Process
Refactored the code then tested to ensure updated sites had their data removed. One think I'm seeing is that activating a site again does not cause it to be reindexed again.
Checklist:
Applicable Issues
Resolved #2275
Changelog Entry
Fixed #2275 - ElasticPress will now remove data from Elasticsearch on a multisite network when a site is archived, deleted or marked as spam