#56913 closed enhancement (fixed)
Prime network options in a single cache call using wp_cache_get_multiple
Reported by: | spacedmonkey | Owned by: | spacedmonkey |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Options, Meta APIs | Keywords: | has-patch needs-testing commit |
Focuses: | performance | Cc: |
Description (last modified by )
When wp_load_core_site_options
change the logic to call wp_cache_get_multiple
priming important network options in cache in a single cache call.
Change History (18)
This ticket was mentioned in PR #3528 on WordPress/wordpress-develop by @spacedmonkey.
2 years ago
#1
- Keywords has-patch added
This ticket was mentioned in Slack in #core-multisite by spacedmonkey. View the logs.
2 years ago
@johnjamesjacoby commented on PR #3528:
2 years ago
#3
Desperately want to pre-register these keys using register_meta()
with some new autoload
attribute 😬
@spacedmonkey commented on PR #3528:
2 years ago
#4
Desperately want to pre-register these keys using
register_meta()
with some newautoload
attribute 😬
Where I personally agree, I feel like that is a larger discussion. This ticket is simple and easier to commit. If this get "fixed" by using the meta api down the line then so be it.
@spacedmonkey commented on PR #3528:
2 years ago
#5
@JJJ Question, how around using a autoload attribute work for multisite. This function is called early in the bootstrap. So would likely require register settings in a sunrise or advanced cache file. This feels weird and unlikely to happen.
#6
@
22 months ago
- Milestone changed from Future Release to 6.3
Would love a review of this as part of 6.3. Pinging @flixos90
#9
@
20 months ago
Now that [55146] is merged, fileupload_maxk
, upload_space_check_disabled
and upload_filetypes
no longer needed to be automatically loaded.
This PR is now just to prime caches in object caches in a single call. There is less benefit, but it still useful. Worth committing @mukesh27 @johnjamesjacoby ?
#10
@
18 months ago
@spacedmonkey you have a failed Unit-test, could you please have a look and fix it. We are in 9 days until Beta 1 🤷
#12
@
18 months ago
- Description modified (diff)
- Summary changed from Improve autoloaded network options to Prime network options in a single cache call using wp_cache_get_multiple
This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.
18 months ago
#14
@
18 months ago
- Keywords needs-testing added
There is a discussion on the PR itself about improvements.
I tested this patch, it works, we have a time improvement with persistent cache. My suggestion is to commit this like it is now and continue discussion for further improvements separately.
Of course, other testers are welcome to test too.
#15
@
18 months ago
- Keywords commit added
I got enough approvals on the PR to commit. I think the discussion should continue as part of the beta process and we can revert if needed.
Trac ticket: https://core.trac.wordpress.org/ticket/56913