Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge 3.8.x into 4.0.x #6218

Merged
merged 11 commits into from
Nov 11, 2023
Merged

Merge 3.8.x into 4.0.x #6218

merged 11 commits into from
Nov 11, 2023

Conversation

derrabus
Copy link
Member

No description provided.

ausi and others added 10 commits November 5, 2023 20:21
|      Q       |   A
|------------- | -----------
| Type         | bug
| Fixed issues | contao/contao#6409

#### Summary

Starting with version 3.7.0 we noticed that the `listTableColumns()`
method got very slow on some systems, our migrate command went from
taking around 2 seconds with version 3.6 to 260 seconds with version3.7.

This is due to the `LEFT JOIN information_schema.CHECK_CONSTRAINTS` not
getting optimized by MariaDB which causes the database server to scan
every table of every database.

This pull request fixes that by using a subquery that is limited by
`WHERE CONSTRAINT_SCHEMA = $databaseName`. This brings the performance
back to where it was with version 3.6.
* 3.7.x:
  Fix MariaDB list columns performance (doctrine#6202)
…#6213)

|      Q       |   A
|------------- | -----------
| Type         | improvement
| Fixed issues |
doctrine#6202 (comment)

#### Summary

Deprecates `AbstractMySQLPlatform::getColumnTypeSQLSnippets()`

~~Needs to be rebased after doctrine#6202
got merged upstream.~~ Done.

---------

Co-authored-by: Alexander M. Turek <[email protected]>
…n layer (doctrine#6215)

|      Q       |   A
|------------- | -----------
| Type         | improvement
| Fixed issues | Follows doctrine#6213

#### Summary

We want to stop guessing the current database.

cc @ausi
|      Q       |   A
|------------- | -----------
| Type         | improvement
| Fixed issues | n/a

#### Summary

Latest Oracle database compatibility should be tested.

Targetting lowest open branch to assert the compatibility for 3.7.x too
for possible bugfixes.
* 3.7.x:
  Add Oracle v23 to CI testing (doctrine#6217)
…options

Enable skipping locked rows in QueryBuilder
@derrabus
Copy link
Member Author

DockerHub seems to have a little hiccup at the moment.

* 3.8.x:
  Enable skipping locked rows in QueryBuilder
  Fix coding standard violation
  Remove unused unit test fixtures
  Add Oracle v23 to CI testing (doctrine#6217)
  Extend `AbstractMySQLPlatform::getColumnTypeSQLSnippets()` deprecation layer (doctrine#6215)
  Deprecate AbstractMySQLPlatform::getColumnTypeSQLSnippets() (doctrine#6213)
  Fix MariaDB list columns performance (doctrine#6202)
@derrabus derrabus merged commit 304e3ec into doctrine:4.0.x Nov 11, 2023
74 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants