Bug #49569

Use deterministic join aliases

Added by Sebastian Kurfuerst about 2 years ago. Updated almost 2 years ago.

Status:Resolved Start date:2013-07-02
Priority:Could have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%

Category:-
Target version:TYPO3 Flow Base Distribution - 2.0.1
PHP Version: Complexity:
Has patch:No Affected Flow version:Flow 2.0.0 beta 2

Description

By using uniqid(), the DQL queries are not properly cached...

Associated revisions

Revision 4b2e5ca6
Added by Sebastian Kurfuerst about 2 years ago

[BUGFIX] Use deterministic join aliases

Before this change, we used uniqid() for building join aliases. This prevented
the doctrine DQL cache to work correctly.

We found this change during profiling TYPO3 Neos, where it led to about
10% performance increase in frontend rendering.

Change-Id: I2af0b81f2c919a66a9d91163294b20a9ccb78fdd
Releases: 2.0, master
Resolves: #49569

Revision 95032c6a
Added by Sebastian Kurfuerst almost 2 years ago

[BUGFIX] Use deterministic join aliases

Before this change, we used uniqid() for building join aliases. This prevented
the doctrine DQL cache to work correctly.

We found this change during profiling TYPO3 Neos, where it led to about
10% performance increase in frontend rendering.

Change-Id: I2af0b81f2c919a66a9d91163294b20a9ccb78fdd
Releases: 2.0, master
Resolves: #49569

History

#1 Updated by Gerrit Code Review about 2 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/21780

#2 Updated by Sebastian Kurfuerst about 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#3 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch 2.0 has been pushed to the review server.
It is available at https://review.typo3.org/22814

#4 Updated by Sebastian Kurfuerst almost 2 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF