Skip to content
Snippets Groups Projects

Feature/transients

Merged Feature/transients
feature/transients into master
Merged Grzegorz Rola requested to merge feature/transients into master

Taka wizja jest moja 😄

Jeżeli transienty mają expiration na 0, to są ładowane tym głównym selectem wraz z wszystkimi opcjami - mają ustawiony autoload na 'yes'. Dodatkowo podczas aktualizacji takich transientów nie ma delete, więc eliminujemy te deadlocki. https://developer.wordpress.org/reference/functions/set_transient/

Dodatkowo ustawiłem aby aktualizacja danych wtyczek była co 5 min a nie co 16 sek.

U klienta tak objawiają się te deadlocki:

[STDERR] WordPress database error Deadlock found when trying to get lock; try restarting transaction for query DELETE FROM wp_gz2dmj_options WHERE option_name = '_transient_require_plugins_data' made by require('wp-blog-header.php'), require_once('wp-load.php'), require_once('wp-config.php'), require_once('wp-settings.php'), include_once('/plugins/flexible-shipping-usps-pro/flexible-shipping-usps-pro.php'), require('/plugins/flexible-shipping-usps-pro/vendor_prefixed/wpdesk/wp-plugin-flow/src/plugin-init-php52.php'), require('/plugins/flexible-shipping-usps-pro/vendor_prefixed/wpdesk/wp-plugin-flow-common/src/plugin-init-php52.php'), FlexibleShippingUspsProVendor\WPDesk\Plugin\Flow\PluginBootstrap->run, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker_With_Update_Disable->are_requirements_met, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->are_requirements_met, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->prepare_requirement_notices, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->check_minimum_require_plugins_version_and_append_notices, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker->retrieve_required_plugins_data, FlexibleShippingUspsProVendor\WPDesk_Basic_Requirement_Checker::retrieve_plugins_data_in_transient, get_transient, delete_option\n

Edited by Grzegorz Rola

Merge request reports

Pipeline #213614 passed

Stage: prepare-vendor
    Stage: tools
      Stage: tests
        Stage: deploy

          Pipeline passed for fdfee628 on feature/transients

          Merged by Grzegorz RolaGrzegorz Rola 1 year ago (Jun 22, 2023 10:45am UTC)

          Loading

          Pipeline #213684 passed

          Stage: prepare-vendor
            Stage: tools
              Stage: tests
                Stage: deploy

                  Pipeline passed for c7af1af2 on master

                  Test coverage 70.98% from 0 jobs
                  Deployed to produ‎ction‎ 1 year ago

                  Activity

                  Filter activity
                  • Approvals
                  • Assignees & reviewers
                  • Comments (from bots)
                  • Comments (from users)
                  • Commits & branches
                  • Edits
                  • Labels
                  • Lock status
                  • Mentions
                  • Merge request status
                  • Tracking
                • Sebastian Pisula removed review request for @sebastian.pisula

                  removed review request for @sebastian.pisula

                • Grzegorz Rola added 1 commit

                  added 1 commit

                  • 4e252dd9 - feature(transients): as autoloaded options

                  Compare with previous version

                • Grzegorz Rola added 1 commit

                  added 1 commit

                  • ca538eac - feature(transients): as autoloaded options

                  Compare with previous version

                • Grzegorz Rola added 1 commit

                  added 1 commit

                  • fdfee628 - feature(transients): as autoloaded options

                  Compare with previous version

                • Grzegorz Rola requested review from @sebastian.pisula

                  requested review from @sebastian.pisula

                • Sebastian Pisula resolved all threads

                  resolved all threads

                • Sebastian Pisula approved this merge request

                  approved this merge request

                • Grzegorz Rola mentioned in commit c7af1af2

                  mentioned in commit c7af1af2

                • Grzegorz Rola merged

                  merged

                • Loading
                • Loading
                • Loading
                • Loading
                • Loading
                • Loading
                • Loading
                • Loading
                • Loading
                • Loading
                  • Please register or sign in to reply