Feature #65460

Create configuration for fields to exclude from ReferenceIndex

Added by Stephan Großberndt over 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2015-03-02
Due date:
% Done:

0%

PHP Version:
Tags:
Complexity:
medium
Sprint Focus:

Description

The patch https://review.typo3.org/#/c/35191 moves the fields formerly hardcoded in getRelations() to a class member to improve performance.

Christian Kuhn and Frans Saris would like to have a configurable solution instead.

When implementing this: Beware a new instance of ReferenceIndex is currently being instanciated for each record! So a configurable solution must provide these fields very fast, best might be to introduce a cache too.


Related issues

Related to TYPO3 Core - Feature #65461: Create configuration for tables to exclude from ReferenceIndex Closed 2015-03-02
Related to TYPO3 Core - Feature #79387: Move access to $nonRelationTables and $nonRelationFields into methods and introduce 2 signals Closed 2017-01-19

History

#1 Updated by Thomas Hohn over 2 years ago

Do we still wan't this or is it sufficient on table level? If so please close the issue

#2 Updated by Stephan Großberndt over 2 years ago

  • Status changed from New to Closed

In #79387 - https://review.typo3.org/#/c/51362/11/typo3/sysext/core/Classes/Database/ReferenceIndex.php we removed the TODO.

Defining columns which are excluded from the reference index in ALL tables of an installation does no longer seem to be a good idea:
Patches to the core always can easily extend/change that column list, extensions should not.

In general it might be an idea to define certain fields inside a specific table to be excluded, but I currently see no use case.
For now I think its sufficient on table level. Should this topic arise again reopen this issue.

Also available in: Atom PDF