Bug #102174: Unnecessary SELECT DATABASE() due to Connection::ensureDatabaseValueTypes() - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #102174

closed

Unnecessary SELECT DATABASE() due to Connection::ensureDatabaseValueTypes()

Added by Christoph Lehmann about 1 year ago. Updated 5 months ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Performance

Target version:

Start date:

2023-10-15

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

When all database values handed over to Connection::ensureDatabaseValueTypes() have types, then it doesn't need to do anything, especially the instantiation of SchemaInformation, whose constructor does the SELECT DATABASE query.

Files

select-database-statements.png (95.3 KB) select-database-statements.png

Christoph Lehmann, 2023-10-15 20:08

Actions

Copy link

Updated by Christoph Lehmann about 1 year ago

File select-database-stacktrace.png added
File select-database-statements.png select-database-statements.png added

Actions

Copy link

Updated by Christoph Lehmann about 1 year ago

File deleted (~~select-database-statements.png~~)

Actions

Copy link

Updated by Christoph Lehmann about 1 year ago

File deleted (~~select-database-stacktrace.png~~)

Actions

Copy link

Updated by Christoph Lehmann about 1 year ago

File deleted (~~select-database-stacktrace.png~~)

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#10

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#11

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#12

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#13

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#14