• Resolved pizzadeliveryguy

    (@pizzadeliveryguy)


    On photo below you can see how plugin makes 48 identical queries in database as SHOW TABLES LIKE 'wp_64121_accessibility_checker'. Every query takes 1,7 second, all together 80 seconds.

    https://pasteboard.co/C5PcfEQOlizP.png

    To try plugin on another server I made an multisite instance with localwp with wordpress 6.9, PHP 8.3.23, MySQL 8.0.35, apache web server.

    Plugin made 48 queries as well such as SHOW TABLES LIKE 'wp_accessibility_checker', every query takes between 0,0001s and 0,0002s, all together 0,0075s.

    The conclusion is when you have huge multisite installation with a large database and lot of tables, query plugins makes become extremely slow and unusable.

    Further reads: https://bugs.mysql.com/bug.php?id=109441

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Author williampatton

    (@williampatton)

    Hey @pizzadeliveryguy,

    Thanks for reporting this and sorry that you are running into this. I will take a look at this today and see where we can reduce the number of times queries run. Seeing this fire 48 times per request seems like a lot to me. Quickly checking on some random pages in my local (not a multisite), I don’t see anywhere close to that number, so I am making an assumption that it’s a multisite-specific issue. That might not be the case but I first investigate it there.

    Is there anything else that you can share that might help me replicate a similar scenario as you see on your own site? Like how many multisites are in there, which plugins are active on most sites or anything else you think might be relevant?

    Thanks again for letting us know, I will keep you updated as I investigate and try to fix this for you.

    Thread Starter pizzadeliveryguy

    (@pizzadeliveryguy)

    Hi,

    I was able to recreate same scenario in both single site and multisite installations. With 2 activated plugins: Accessibility checker and Query monitor.

    In first post there are specifications of database, php and such. There was no other modifications.

    Plugin Author williampatton

    (@williampatton)

    Hey again @pizzadeliveryguy

    I was able to reproduce this when I visit the options page for Accessibility Checker. I can see a way to make this much more efficient and reduce the number of calls it makes a lot.

    Plugin Author williampatton

    (@williampatton)

    Hi @pizzadeliveryguy

    I have a solution that reduces the number of times this needs to get called down to just 2 times rather than 48. The 48 times it runs happens when the site stats regenerate (first time you visit the welcome page and again after 24 hours when the site stats cache clears).

    We will release that change in our next plugin release (likely next Tuesday). If you want to get it earlier than that you can visit our github and I can build you a pre-release there. The issue I am tracking this on is: https://github.com/equalizedigital/accessibility-checker/issues/1295

    I am also looking into a method to prevent needing to parse the entire table list as well but not entirely certain if/when we will release that.

    Thread Starter pizzadeliveryguy

    (@pizzadeliveryguy)

    Hey,

    thank you for fast response quick resolution! I try out your new release.

    Plugin Author williampatton

    (@williampatton)

    Hey again @pizzadeliveryguy,

    Just wanted to let you know that we released an update for the plugin today to verion 1.35.0 which included a fix for this. It reduces the number of times that has to run on our welcome page to a max of 2 times. We will look to reduce it to just 1 soon or maybe even get rid of it entirely and swap to a more performant query to verify our table exist.

    Thanks again for reporting the issue, if you have any other problems let us know.

    Once you have had a change to test the plugin more it would be great if you could lave a review of the plugin for us here in the ww.wp.xz.cn site. Those reviews really help us out a lot.

    And enjoy the weekend!

    Thread Starter pizzadeliveryguy

    (@pizzadeliveryguy)

    Hi, thank you for a quick! I believe now it’s working as supposed to.

Viewing 7 replies - 1 through 7 (of 7 total)

You must be logged in to reply to this topic.