Bug #34886
Task #52304: Performance issues
t3lib_cache_backend_FileBackend don't support 0 has liftime
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
Associated revisions
[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
[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
Applied in changeset c8d2033e3a0c470d8e22e9c1ed830367f0aa1849.
#15 Updated by Jigal van Hemert over 1 year ago
- Is Regression changed from No to Yes