[PR #372] [DDC-1845] QuoteStrategy #8096

Open
opened 2026-01-22 15:58:22 +01:00 by admin · 0 comments
Owner

Original Pull Request: https://github.com/doctrine/orm/pull/372

State: closed
Merged: Yes


QuoteStrategy

http://www.doctrine-project.org/jira/browse/DDC-1845

This patch fix some quote problems using a default quote strategy and allows users find solutions themselves for weird quote cases.

This DBAL PR shoud be merged to fix sqlite tests in : https://github.com/doctrine/dbal/pull/158

There is a lote of new method calls, the performance tests in sqlite are the following :

My branch :

. testSimpleQueryScalarHydrationPerformance10000Rows - 0.37292098999023 seconds
. testSimpleQueryArrayHydrationPerformance10000Rows - 0.50411009788513 seconds
. testMixedQueryFetchJoinArrayHydrationPerformance10000Rows - 1.1323919296265 seconds
. testSimpleQueryPartialObjectHydrationPerformance10000Rows - 1.0243051052094 seconds
. testSimpleQueryFullObjectHydrationPerformance10000Rows - 4.1673181056976 seconds
. testMixedQueryFetchJoinPartialObjectHydrationPerformance2000Rows - 0.40403699874878 seconds
. testMixedQueryFetchJoinFullObjectHydrationPerformance2000Rows - 0.873291015625 seconds
. 99 CompanyContract: 0.023395
. 99 CompanyContract: 0.019528
. Memory usage before: 111715.53125 KB
. Memory usage after: 110282.875 KB
. Inserted 10000 objects in 5.5240259170532 seconds
. 100 CmsArticle findAll(): 0.018410
. 100 CmsArticle findAll(): 0.014157
. 100 CmsArticle find(): 0.043304
. 100 CmsArticle find(): 0.041451
. 100 CmsGroup: 0.009315
. 100 CmsGroup: 0.009239
. 100 CmsUser: 0.024051
. 100 CmsUser: 0.023655
. Compute ChangeSet 100 objects in 0.040482997894287 seconds

. Time: 19 seconds, Memory: 261.50Mb

Doctrine Master :

. testSimpleQueryScalarHydrationPerformance10000Rows - 0.37668490409851 seconds
. testSimpleQueryArrayHydrationPerformance10000Rows - 0.5009880065918 seconds
. testMixedQueryFetchJoinArrayHydrationPerformance10000Rows - 1.131824016571 seconds
. testSimpleQueryPartialObjectHydrationPerformance10000Rows - 1.037791967392 seconds
. testSimpleQueryFullObjectHydrationPerformance10000Rows - 4.1992859840393 seconds
. testMixedQueryFetchJoinPartialObjectHydrationPerformance2000Rows - 0.41158103942871 seconds
. testMixedQueryFetchJoinFullObjectHydrationPerformance2000Rows - 0.86575293540955 seconds
. 99 CompanyContract: 0.023206
. 99 CompanyContract: 0.020555
. Inserted 10000 objects in 5.4995818138123 seconds
. 100 CmsArticle findAll(): 0.018785
. 100 CmsArticle findAll(): 0.014160
. 100 CmsArticle find(): 0.042067
. 100 CmsArticle find(): 0.041076
. 100 CmsGroup: 0.009478
. 100 CmsGroup: 0.010591
. 100 CmsUser: 0.023727
. 100 CmsUser: 0.023073
. Compute ChangeSet 100 objects in 0.040009021759033 seconds

. Time: 19 seconds, Memory: 260.50Mb
**Original Pull Request:** https://github.com/doctrine/orm/pull/372 **State:** closed **Merged:** Yes --- ### QuoteStrategy http://www.doctrine-project.org/jira/browse/DDC-1845 This patch fix some quote problems using a default quote strategy and allows users find solutions themselves for weird quote cases. This DBAL PR shoud be merged to fix sqlite tests in : https://github.com/doctrine/dbal/pull/158 There is a lote of new method calls, the performance tests in sqlite are the following : #### My branch : ``` . testSimpleQueryScalarHydrationPerformance10000Rows - 0.37292098999023 seconds . testSimpleQueryArrayHydrationPerformance10000Rows - 0.50411009788513 seconds . testMixedQueryFetchJoinArrayHydrationPerformance10000Rows - 1.1323919296265 seconds . testSimpleQueryPartialObjectHydrationPerformance10000Rows - 1.0243051052094 seconds . testSimpleQueryFullObjectHydrationPerformance10000Rows - 4.1673181056976 seconds . testMixedQueryFetchJoinPartialObjectHydrationPerformance2000Rows - 0.40403699874878 seconds . testMixedQueryFetchJoinFullObjectHydrationPerformance2000Rows - 0.873291015625 seconds . 99 CompanyContract: 0.023395 . 99 CompanyContract: 0.019528 . Memory usage before: 111715.53125 KB . Memory usage after: 110282.875 KB . Inserted 10000 objects in 5.5240259170532 seconds . 100 CmsArticle findAll(): 0.018410 . 100 CmsArticle findAll(): 0.014157 . 100 CmsArticle find(): 0.043304 . 100 CmsArticle find(): 0.041451 . 100 CmsGroup: 0.009315 . 100 CmsGroup: 0.009239 . 100 CmsUser: 0.024051 . 100 CmsUser: 0.023655 . Compute ChangeSet 100 objects in 0.040482997894287 seconds . Time: 19 seconds, Memory: 261.50Mb ``` #### Doctrine Master : ``` . testSimpleQueryScalarHydrationPerformance10000Rows - 0.37668490409851 seconds . testSimpleQueryArrayHydrationPerformance10000Rows - 0.5009880065918 seconds . testMixedQueryFetchJoinArrayHydrationPerformance10000Rows - 1.131824016571 seconds . testSimpleQueryPartialObjectHydrationPerformance10000Rows - 1.037791967392 seconds . testSimpleQueryFullObjectHydrationPerformance10000Rows - 4.1992859840393 seconds . testMixedQueryFetchJoinPartialObjectHydrationPerformance2000Rows - 0.41158103942871 seconds . testMixedQueryFetchJoinFullObjectHydrationPerformance2000Rows - 0.86575293540955 seconds . 99 CompanyContract: 0.023206 . 99 CompanyContract: 0.020555 . Inserted 10000 objects in 5.4995818138123 seconds . 100 CmsArticle findAll(): 0.018785 . 100 CmsArticle findAll(): 0.014160 . 100 CmsArticle find(): 0.042067 . 100 CmsArticle find(): 0.041076 . 100 CmsGroup: 0.009478 . 100 CmsGroup: 0.010591 . 100 CmsUser: 0.023727 . 100 CmsUser: 0.023073 . Compute ChangeSet 100 objects in 0.040009021759033 seconds . Time: 19 seconds, Memory: 260.50Mb ```
admin added the pull-request label 2026-01-22 15:58:22 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#8096