Project

General

Profile

Actions

Bug #25119

closed

performace of cachingframework

Added by Björn Pedersen over 13 years ago. Updated about 11 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2011-06-20
Due date:
2011-06-20
% Done:

100%

Estimated time:
TYPO3 Version:
4.5
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

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

Related issues 4 (0 open4 closed)

Related to TYPO3 Core - Task #26263: Update caching framework to latest FLOW3 versionClosedChristian Kuhn2011-04-25

Actions
Follows TYPO3 Core - Bug #27506: Caching framework: Flush all caches with 'clear all cache' commandClosed2011-06-17

Actions
Follows TYPO3 Core - Task #27536: Caching framework db backend: Use own namespaceClosed2011-06-19

Actions
Precedes TYPO3 Core - Bug #27548: Caching framework db backend: Remove wildcard tag droppingClosed2011-06-212011-06-21

Actions
Actions #1

Updated by Christian Kuhn about 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?

Actions #2

Updated by Björn Pedersen about 13 years ago

Yes, that idea sounds more or less OK.

Wether I really find time I can not promise right now.

Actions #3

Updated by Mr. Hudson about 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

Actions #4

Updated by Björn Pedersen about 13 years ago

The new patch now implements the table auto-creation.

Actions #5

Updated by Mr. Hudson about 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

Actions #6

Updated by Mr. Hudson about 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

Actions #7

Updated by Mr. Hudson about 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

Actions #8

Updated by Mr. Hudson about 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

Actions #9

Updated by Mr. Hudson about 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

Actions #10

Updated by Mr. Hudson about 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

Actions #11

Updated by Mr. Hudson about 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

Actions #12

Updated by Mr. Hudson about 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

Actions #13

Updated by Mr. Hudson about 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

Actions #14

Updated by Björn Pedersen about 13 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100
Actions #15

Updated by Steffen Gebert about 13 years ago

  • Target version set to 1281
Actions #16

Updated by Xavier Perseguers over 12 years ago

  • Status changed from Resolved to Closed
Actions #17

Updated by Ernesto Baschny about 11 years ago

  • Target version deleted (1281)
Actions

Also available in: Atom PDF