View Issue Details

IDProjectCategoryView StatusLast Update
0000284Adventure PHP FrameworkCode-Verbesserung // Code improvementpublic2016-02-28 17:49
ReporterChristianAchatzAssigned ToChristianAchatz 
PrioritynormalSeverityfeatureReproducibilityN/A
Status resolvedResolutionfixed 
Product Version3.1 
Target Version3.2Fixed in Version3.2 
Summary0000284: Optimize performance for filling place holder
DescriptionATM, filling 10000 non-existing place holders takes around 7s. This is due to exception handling in setPlaceHolder() and implicitly also in setPlaceHolders().

Numbers are increasing when using setPlaceHolderIfExists() or setPlaceHoldersIfExist().

Within this issue place holder setting performance should be improved by the following measures:

- make throwing exceptions with non-existing place holders optional by static configuration in Document (proposal: $beStrictWithPlaceHolders = true|false)
- remove methods setPlaceHolderIfExists() and setPlaceHoldersIfExist()
- remove exception handling and logging (makes things even worse)
TagsNo tags attached.
Codereferenz: ([Datei]:[Zeile])
Namespacecore

Activities

ChristianAchatz

2016-02-28 17:34

administrator   ~0000675

Last edited: 2016-02-28 17:49

View 2 revisions

- Improved performance from ~ 7s to 0.05s for 100000 iterations.
- Added migration script to remove *IfExists() methods.
- Added migration documentation with additional hint.

Issue History

Date Modified Username Field Change
2016-02-28 17:15 ChristianAchatz New Issue
2016-02-28 17:15 ChristianAchatz Status new => assigned
2016-02-28 17:15 ChristianAchatz Assigned To => ChristianAchatz
2016-02-28 17:34 ChristianAchatz Note Added: 0000675
2016-02-28 17:34 ChristianAchatz Status assigned => resolved
2016-02-28 17:34 ChristianAchatz Fixed in Version => 3.2
2016-02-28 17:34 ChristianAchatz Resolution open => fixed
2016-02-28 17:49 ChristianAchatz Note Edited: 0000675 View Revisions