Feature #48960

Epic #55070: Workpackages

Epic #55065: WP: Overall System Performance (Backend and Frontend)

Epic #55656: Optimize overall Extbase performance

Extbase: Backport lazy dependency injection

Added by Markus Günther over 6 years ago. Updated about 2 years ago.

Status:
Rejected
Priority:
Should have
Category:
Extbase
Target version:
-
Start date:
2013-06-08
Due date:
% Done:

0%

PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

In TYPO3 flow 2.0 we have now lazy dependency injection. This feature is really awesome and it will be cool if we have this also in extbase. In TYPO3 flow this feature speed up flow because the the injected classes will only loaded if you need them.

The related flow ticket is http://forge.typo3.org/issues/43819


Related issues

Related to TYPO3.Flow - Feature #43819: Support lazy dependency injection of properties Resolved 2012-12-09

History

#1 Updated by Markus Günther over 6 years ago

The backport from the feature is not as easy as i expected, because the implementation of DI is completely different in TYPO3 flow. So if we want to have this the migration will be a little bit complex.

#2 Updated by Anja Leichsenring over 6 years ago

  • Target version changed from 6.2.0 to Extbase 6.2

#3 Updated by Alexander Schnitzler over 6 years ago

So, are you working on it at all? If not let's flag this 6.3.

#4 Updated by Markus Günther over 6 years ago

  • Target version changed from Extbase 6.2 to Extbase 6.3

#5 Updated by Gerrit Code Review over 5 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27001

#6 Updated by Gerrit Code Review over 5 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27001

#7 Updated by Gerrit Code Review over 5 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27001

#8 Updated by Gerrit Code Review over 5 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27001

#9 Updated by Marc Bastian Heinrichs over 5 years ago

  • Assignee set to Marc Bastian Heinrichs
  • Parent task set to #55656

#10 Updated by Felix Oertel over 5 years ago

Do you have some numbers on this? I don't think, that a real performance boost might apply to our situation as it does to Flow.

#11 Updated by Gerrit Code Review over 5 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27001

#12 Updated by Marc Bastian Heinrichs over 5 years ago

Will count some xhprof numbers

#13 Updated by Ernesto Baschny over 5 years ago

  • Project changed from Extbase MVC Framework to TYPO3 Core
  • Target version deleted (Extbase 6.3)

#14 Updated by Ernesto Baschny over 5 years ago

  • Subject changed from Backport lazy dependency injection to Extbase: Backport lazy dependency injection
  • Category set to Extbase
  • TYPO3 Version set to 6.2

#15 Updated by Christian Kuhn over 4 years ago

  • Status changed from Under Review to New

#16 Updated by Christian Kuhn over 4 years ago

The patch pushed to gerrit was abandoned since it was stalled for a long time. This needs a fresh approach :)

#17 Updated by Benni Mack about 2 years ago

  • Status changed from New to Rejected

not gonna happen within Extbase scope (if we do DI then we'd go all in for the whole core)

Also available in: Atom PDF