DDC-1870: [GH-372] [DDC-1845] QuoteStrategy #2360

Closed
opened 2026-01-22 13:49:37 +01:00 by admin · 4 comments
Owner

Originally created by @doctrinebot on GitHub (Jun 12, 2012).

Originally assigned to: @beberlei on GitHub.

Jira issue originally created by user @beberlei:

This issue is created automatically through a Github pull request on behalf of FabioBatSilva:

Url: https://github.com/doctrine/doctrine2/pull/372

Message:

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
Originally created by @doctrinebot on GitHub (Jun 12, 2012). Originally assigned to: @beberlei on GitHub. Jira issue originally created by user @beberlei: This issue is created automatically through a Github pull request on behalf of FabioBatSilva: Url: https://github.com/doctrine/doctrine2/pull/372 Message: ### QuoteStrategy http://www.doctrine-project.org/jira/browse/[DDC-1845](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 Bug label 2026-01-22 13:49:37 +01:00
admin closed this issue 2026-01-22 13:49:39 +01:00
Author
Owner

@doctrinebot commented on GitHub (Jun 25, 2012):

Comment created by @beberlei:

A related Github Pull-Request [GH-372] was closed
https://github.com/doctrine/doctrine2/pull/372

@doctrinebot commented on GitHub (Jun 25, 2012): Comment created by @beberlei: A related Github Pull-Request [GH-372] was closed https://github.com/doctrine/doctrine2/pull/372
Author
Owner

@doctrinebot commented on GitHub (Jul 4, 2012):

Issue was closed with resolution "Fixed"

@doctrinebot commented on GitHub (Jul 4, 2012): Issue was closed with resolution "Fixed"
Author
Owner

@doctrinebot commented on GitHub (Sep 15, 2013):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-372] was closed:
https://github.com/doctrine/dbal/pull/372

@doctrinebot commented on GitHub (Sep 15, 2013): Comment created by @doctrinebot: A related Github Pull-Request [GH-372] was closed: https://github.com/doctrine/dbal/pull/372
Author
Owner

@doctrinebot commented on GitHub (Sep 22, 2014):

Comment created by @doctrinebot:

A related Github Pull-Request [GH-372] was assigned:
https://github.com/doctrine/dbal/pull/372

@doctrinebot commented on GitHub (Sep 22, 2014): Comment created by @doctrinebot: A related Github Pull-Request [GH-372] was assigned: https://github.com/doctrine/dbal/pull/372
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: doctrine/archived-orm#2360