-
Notifications
You must be signed in to change notification settings - Fork 891
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Feature Request] Provide the ability to completely disable text link counting. #9474
Comments
Please inform the customer of conversation # 343730 when this conversation has been closed. |
Is there an ETA as to when this issue will be resolved? If not, is there a workaround / temp fix to stop records from being inserted into |
@jdevalk There are some decisions that need to be made around the flow. There are multiple use cases:
Questions:
|
See #12751 (comment) for the conclusions about the questions above. |
Disabling the Text Link Counter feature will still cause links in content to be analyzed and inserted in the "_yoast_seo_links" table. We've received a request for a way (toggle or filter) to completely disable this, since this feature might be disabled on large sites for performance reasons.
Report from the user:
On post save, in wordpress-seo-premium/admin/links/class-link-content-processor.php - WPSEO_Link_Content_Processor::process() it counts links and calls WPSEO_Link_Storage::save_links() without bothering to check the setting. In our case, posts with 200+ links are invoking 200+ INSERTs into the yoast_seo_links table. Due to replication in our current setup, this can take seconds per INSERT query which causes a massive performance issue. Luckily WPSEO_Link_Table_Accessible can be fooled into aborting if the table is renamed, but ideally either the Admin setting, or a dedicated apply_filters could disable this functionality.
The text was updated successfully, but these errors were encountered: