Bug #34886

Task #52304: Performance issues

t3lib_cache_backend_FileBackend don't support 0 has liftime

Added by Dominique Feyer over 3 years ago. Updated over 1 year ago.

Status:Resolved Start date:2012-03-15
Priority:Should have Due date:
Assigned To:Dominique Feyer % Done:

100%

Category:Caching Spent time: -
Target version:-
TYPO3 Version:4.7 Is Regression:Yes
PHP Version: Sprint Focus:
Complexity:

Description

In the DB backend 0 has liftime mean "unlimited" lifetime for the cache entry, but in the file backend it's give a always invalid cache entry.

Patch is coming


Related issues

related to Core - Feature #39430: Update cache framework to latest FLOW3 version Resolved 2014-04-17
related to Core - Bug #52125: Saving records takes ages to complete Resolved 2013-09-19
related to Core - Bug #58008: Autoload cache never expires and is not flushed after ins... Closed 2014-04-17

Associated revisions

Revision e06f05a6
Added by Dominique Feyer almost 2 years ago

[BUGFIX] CF FileBackend unlimited lifetime support

Backport of a bugfix that was part of #39430 in 6.0.

Change-Id: I2266ae12284a139a384854e3ff0bacf23f277859
Resolves: #34886
Related: #39430
Releases: 4.7, 4.5
Reviewed-on: https://review.typo3.org/24073
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Revision c8d2033e
Added by Dominique Feyer almost 2 years ago

[BUGFIX] CF FileBackend unlimited lifetime support

Backport of a bugfix that was part of #39430 in 6.0.

Change-Id: I2266ae12284a139a384854e3ff0bacf23f277859
Resolves: #34886
Related: #39430
Releases: 4.7, 4.5
Reviewed-on: https://review.typo3.org/24073
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit e06f05a67c6c516f5970ce4d5785f8c9356e34ff)
Reviewed-on: https://review.typo3.org/24083

History

#1 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

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

#2 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9671

#3 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9671

#4 Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9671

#5 Updated by Oliver Hader over 3 years ago

Thanks for your patch. How was the behaviour in 4.5 (since your patch only is reported for 4.6 and 4.7). Is it an issue in 4.5 as well?

#6 Updated by Dominique Feyer over 3 years ago

Yes it's an issues in 4.5 also

#7 Updated by Gerrit Code Review about 3 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9671

#8 Updated by Alexander Opitz about 2 years ago

Hi Dominique,

is this patch still relevant? If yes, can you rebase the patch to current master and add a bit more description to your commit message?

#9 Updated by Christian Kuhn almost 2 years ago

  • Parent task set to #52304
  • Is Regression set to No

#10 Updated by Christian Kuhn almost 2 years ago

This is not an issue in 6.0, it was fixed with #39430

I'm unsure if this issue is still important enough for 4.7 / 4.5

#11 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/24073

#12 Updated by Gerrit Code Review almost 2 years ago

Patch set 4 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/24073

#13 Updated by Gerrit Code Review almost 2 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/24083

#14 Updated by Dominique Feyer almost 2 years ago

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

#15 Updated by Jigal van Hemert over 1 year ago

  • Is Regression changed from No to Yes

Also available in: Atom PDF