We read all the configuration data in StorageComparer and statically cache in that object. This means that any events and the importer have to reread the configuration.
We can wrap the source and target storages with
CachedStorage so that we don't have to re-read the data.
User interface changes
Beta phase evaluation
|Issue category||Task because it not fixing a bug - just a small internal refactor of StorageComparer|
|Issue priority||Major because during configuration import we read a lot of data from disk and the db anything we can do to minimise the amount of times we have to read from disk or the db is a good thing. Additionally as we add configuration import validators they will need to interact with incoming data - we shouldn't be reading everything twice.|
|Prioritized changes||The main goal of this issue is performance.|
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 92,430 pass(es). View
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 2411689.14.patch. Unable to apply patch. See the log in the details link for more information. View
FAILED: [[SimpleTest]]: [PHP 5.4 MySQL] Unable to apply patch 2411689.5.patch. Unable to apply patch. See the log in the details link for more information. View
PASSED: [[SimpleTest]]: [PHP 5.4 MySQL] 84,243 pass(es). View