mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 14:05:37 +02:00
FIX Run ANALYZE TABLE before fetching table stats
This commit is contained in:
parent
7b91207c12
commit
25a41de8b5
@ -569,23 +569,21 @@ class MySQLDatabase extends Database implements TransactionManager
|
||||
// Not simply using "TRUNCATE TABLE \"$table\"" because DELETE is a lot quicker
|
||||
// than TRUNCATE which is very relevant during unit testing. Using TRUNCATE will lead to an
|
||||
// approximately 50% increase it the total time of running unit tests.
|
||||
//
|
||||
// Using max(ID) to determine if the table should reset its auto-increment, rather than using
|
||||
// SELECT "AUTO_INCREMENT" FROM INFORMATION_SCHEMA.TABLES WHERE TABLE_SCHEMA = ? AND TABLE_NAME = ?
|
||||
// after deleting from the table, because in MySQL 8, under certain conditions, notably
|
||||
// when running behat, sometimes the auto-increment was being reset to 2 for unknown reasons
|
||||
$self = $this;
|
||||
$fn = function () use ($self, $table) {
|
||||
$maxID = $self->query("SELECT MAX(ID) FROM \"$table\"")->value();
|
||||
$self->query("DELETE FROM \"$table\"");
|
||||
if ($maxID > 0) {
|
||||
$self->query("ALTER TABLE \"$table\" AUTO_INCREMENT = 1");
|
||||
}
|
||||
};
|
||||
if ($this->supportsTransactions()) {
|
||||
$this->withTransaction($fn);
|
||||
} else {
|
||||
$fn();
|
||||
$this->query("DELETE FROM \"$table\"");
|
||||
|
||||
// Check if resetting the auto-increment is needed
|
||||
|
||||
// First run ANALYZE TABLE to reset table stats which are cached for 24 hours by
|
||||
// default in MySQL 8
|
||||
$this->query("ANALYZE TABLE \"$table\"");
|
||||
|
||||
$autoIncrement = $this->preparedQuery(
|
||||
'SELECT "AUTO_INCREMENT" FROM INFORMATION_SCHEMA.TABLES WHERE TABLE_SCHEMA = ? AND TABLE_NAME = ?',
|
||||
[ $this->getSelectedDatabase(), $table]
|
||||
)->value();
|
||||
|
||||
if ($autoIncrement > 1) {
|
||||
$this->query("ALTER TABLE \"$table\" AUTO_INCREMENT = 1");
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user