Project

General

Profile

Actions

Bug #75783

closed

Unoptimized query exceeds memory_limit in lowlevel

Added by Stefan Froemken over 8 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Performance
Target version:
Start date:
2016-04-19
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.5
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

Hello Core-Team,

following command:

/usr/local/bin/php5-55STABLE-CLI /www/123/rp-hosting/123/123/typo3cms/project1/typo3/cli_dispatch.h lowlevel_cleaner missing_relations -r --refindex update

results in memory_limit exceeded.

I have startet a debug session. It has nothing to do with the refindex itself. It has more to do with the "missing_relations" Argument.
This argument starts MissingRelationsCommand which will execute:

$recs = $GLOBALS['TYPO3_DB']->exec_SELECTgetRows('*', 'sys_refindex', 'ref_table<>' . $GLOBALS['TYPO3_DB']->fullQuoteStr('_FILE', 'sys_refindex') . ' AND ref_uid>0' . $filterClause, '', 'sorting DESC');

I have pre- and appended a memory_get_peak_usage(TRUE) which results in:

Before: 15466496 Bytes
After: 104857600 Bytes

I think it's too much to retrieve nearly all table information from sys_refindex at that point.

Stefan

Actions #1

Updated by Markus Klein over 8 years ago

Simply change to sql_fetch() and that will not happen anymore.

Actions #2

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #3

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #4

Updated by Markus Klein over 8 years ago

  • Assignee set to Markus Klein
  • Target version changed from 6.2.20 to next-patchlevel
  • Complexity set to easy
  • Sprint Focus set to Stabilization Sprint
Actions #5

Updated by Gerrit Code Review over 8 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47788

Actions #6

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47788

Actions #7

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #8

Updated by Gerrit Code Review over 8 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #9

Updated by Gerrit Code Review over 8 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #10

Updated by Gerrit Code Review over 8 years ago

Patch set 3 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47788

Actions #11

Updated by Gerrit Code Review over 8 years ago

Patch set 4 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47788

Actions #12

Updated by Gerrit Code Review over 8 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #13

Updated by Gerrit Code Review over 8 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #14

Updated by Gerrit Code Review over 8 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47786

Actions #15

Updated by Markus Klein over 8 years ago

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

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF