-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Performance issues with metadatas #137
Conversation
Codecov Report
@@ Coverage Diff @@
## 2.x #137 +/- ##
==========================================
- Coverage 92.98% 92.95% -0.04%
==========================================
Files 44 44
Lines 1526 1561 +35
==========================================
+ Hits 1419 1451 +32
- Misses 107 110 +3
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Thanks again @dmitryuk! This one was on my roadmap :) |
Test example:
Without cache: With cache: |
* Cache DH Annotations * speed * remove getOwner call Co-authored-by: a.dmitryuk <a.dmitryuk@movavi.com>
* Fix bug: exception while creating audit table schema when using single table inheritance (#133) * fix #132 * entity Co-authored-by: a.dmitryuk <a.dmitryuk@movavi.com> * Updated README.md * Updated CI * Fixed annotations in tests * Regression fix. Fixes DamienHarper/auditor-bundle#334 * Updated CI * Performance issues with metadatas (#137) * Cache DH Annotations * speed * remove getOwner call Co-authored-by: a.dmitryuk <a.dmitryuk@movavi.com> * CreateSchemaListener not updating inheritance tables (#139) * Fix: Attempted to call an undefined method named "getMetadataCache" of class "Doctrine\ORM\Configuration". (#144) * Fix: Attempted to call an undefined method named "introspectSchema" of class "Doctrine\DBAL\Schema\PostgreSQLSchemaManager" (#143) * Fixed invokable storage mapper (see #146) (#148) * Fixed broken annotation registration since doctrine/orm 2.14 * improved the TransactionProcessor to convert encoding of diff properly (#152) * improved the TransactionProcessor to convert encoding of diff properly * adapted quotes in TransactionProcessor to single quotes * Fixed compatibility issue with doctrine/event-manager ^2.0 (#157) * PHP-CS-Fixer * Validate that diff is a string before passing to mb_convert_encoding (#156) * PHP-CS-Fixer * Support doctrine annotations 2.0 (#158) * topic 344 doctrine annotations 2.0 --------- Co-authored-by: a.dmitryuk <a.dmitryuk@movavi.com> * PHP-CS-Fixer * Fix failing CI (lock related) * PHP-CS-Fixer * Fixes `diffs` column type when JSON type is supported * Fixed CI (2.x) * Fixed CI (2.x) * Fixed CI (2.x) * Fixed CI (2.x) * Fixed CI (2.x) * Fix 185 (#186) * fixes #185 * PHP-CS-Fixer --------- Co-authored-by: Damien Harper <damien.harper@gmail.com> * skip embedded class from schema listener (#189) Co-authored-by: Guillaume Sainthillier <guillaume.sainthillier@gmail.com> * Fixed CI (2.x) * Typo * PHP-CS-Fixer * Update deps + PHP-CS-Fixer * PHP-CS-Fixer * PHP-CS-Fixer (3.48.0) * Detected wrapped driver (#195) * - updated DHDriver and use parent::connect instead of dedicated field - added getWrappedDriver to extract wrapped driver. - tests Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * Fixed PHPStan error --------- Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> Co-authored-by: Damien Harper <damien.harper@gmail.com> * fix Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * php-cs-fixer Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * removed getSubscribedEvents from CreateSchemaListener Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * phpstan Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * fix DoctrineSubscriber + tests Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * fix Issue174Test Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> * removed duplicate code, fixed test Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> --------- Signed-off-by: Oleg Andreyev <oleg@andreyev.lv> Co-authored-by: Alexander Dmitryuk <xakzona@bk.ru> Co-authored-by: a.dmitryuk <a.dmitryuk@movavi.com> Co-authored-by: damienharper <damien.harper@gmail.com> Co-authored-by: Jörn Dyherrn <joern@dyherrn.dev> Co-authored-by: Martijn Boers <2955898+martijnboers@users.noreply.github.com> Co-authored-by: Guillaume Sainthillier <guillaume.sainthillier@gmail.com>
Closes DamienHarper/auditor-bundle#319
Metadata cache should be configured to start it working