Bug #25119
closedperformace of cachingframework
Added by Björn Pedersen over 13 years ago. Updated over 11 years ago.
100%
Description
as dicussed in some length on the typo3-dev mailing list,
the cachingframework is not optimized with respect to query indeces.
this patch provides an alternative db_backend with expires-column and indeces for better db usage.
I have choosen a second be class, as otherwise extension using CF would break due to different table layout.
(issue imported from #M17693)
Files
cache_be_db.patch (18 KB) cache_be_db.patch | Administrator Admin, 2011-02-21 16:33 |
Updated by Christian Kuhn over 13 years ago
- Target version deleted (
0)
Idea is to transfer the table creation and layout within the db backend to have control about it on layout changes and to not depend on extensions to adapt layout if we need it in core.
It will propably become tricky in em to handle it fully automatically at this point (with whatever other db changes are lying around).
The best idea I've had until now is to do it in flush():
- We know final table names at this point
- Tables are truncated there anyway, we just always do an additional drop and create, it will be quick
- Caches are flushed on extension change anyway, together with the flow3 cache update patch the cache manager is able to flush() all defined caches, even if they are not instantiated.
- No overhead at runtime / in cache production
- Table names can be determined automatically by db backend, so tagsTable and dataTable settings are optional after the patch.
Until now I did not figured on how to do it best:
- Probably one or two sql resource blueprint files in t3lib/cache/backend/resource/ where we fiddle our current tables names into, or something like that
- The main table creation can probably be done by em sql compare magic (it should handle dbal for us for thinks like the Engin=Innodb line), but I have no idea for the actually needed code and didn't check this until now
I'm unsure if we should create an own issue for this and handle our issue here after we've changed the db backend.
@Björn: Do you have time and fun to hack on this issue?
Updated by Björn Pedersen over 13 years ago
Yes, that idea sounds more or less OK.
Wether I really find time I can not promise right now.
Updated by Mr. Hudson over 13 years ago
Patch set 5 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Björn Pedersen over 13 years ago
The new patch now implements the table auto-creation.
Updated by Mr. Hudson over 13 years ago
Patch set 6 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 7 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 8 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 9 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 10 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 11 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 12 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 13 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Mr. Hudson over 13 years ago
Patch set 14 of change Ie97563e1612d64759bcbd2ee4db005a6d5a33812 has been pushed to the review server.
It is available at http://review.typo3.org/1083
Updated by Björn Pedersen over 13 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 100
Applied in changeset 0f6891581e13e91e195c09cde498c9e137609581.
Updated by Xavier Perseguers over 12 years ago
- Status changed from Resolved to Closed