Added extra layer of caching / reduce load on Redis#19
Open
maximbaibakov wants to merge 1 commit intomagento:masterfrom
Open
Added extra layer of caching / reduce load on Redis#19maximbaibakov wants to merge 1 commit intomagento:masterfrom
maximbaibakov wants to merge 1 commit intomagento:masterfrom
Conversation
Zend_Locale_Data returns a formatted locale string based on the input. Once string is proceed it stored into cache storage File or Redis or Memcache or MySQL. Every time someone is asking for the same formatted locale string, it will get it from cache. However, if the same execution is asking for the same locale string, it always looking into the cache storage. Ideally, we can store result in temporary variable to reduce the load on remote cache storage. Example: - There is a booking website - There are number of providers who have their open hours - Zend_Date function is in use to present data in correct locale - Open hour from each provider going via Zend_Date function to display the right date / locale. - Page has about 20-50 providers, in some cases, the number of calls to cache storage (e.g. Redis) could jump to 22,000+ on page load. This commit introduced a temporary variable to store a cache for a single page load.
|
|
|
Hello @maximbaibakov, Thank you for contribution. We begin to process the PR. |
|
FYI, In OpenMage we have included similar optimization already |
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
Zend_Locale_Data returns a formatted locale string based on the input.
Once string is proceed it stored into cache storage File or Redis or Memcache or MySQL.
Every time someone is asking for the same formatted locale string, it will get it from cache.
However, if the same execution is asking for the same locale string, it always looking into the cache storage. Ideally, we can store result in temporary variable to reduce the load on remote cache storage.
Example:
This commit introduced a temporary variable to store a cache for a single page load.