Merge pull request #19789 from ping-yee/250728-fix-19504

Fix #19504 - set default value for the innodb.

Fixes #19504
This commit is contained in:
Maurício Meneghini Fauth 2025-08-25 18:26:39 -03:00 committed by GitHub
commit 22948b7db2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -396,7 +396,22 @@ class Advisor
*/
private function evaluateRuleExpression(string $expression)
{
return $this->expression->evaluate($expression, array_merge($this->variables, $this->globals));
$variables = array_merge($this->variables, $this->globals);
// Set default values for missing InnoDB variables when InnoDB is disabled
if (! isset($variables['innodb_buffer_pool_size'])) {
$variables['innodb_buffer_pool_size'] = 0;
}
if (! isset($variables['innodb_log_file_size'])) {
$variables['innodb_log_file_size'] = 0;
}
if (! isset($variables['innodb_log_files_in_group'])) {
$variables['innodb_log_files_in_group'] = 0;
}
return $this->expression->evaluate($expression, $variables);
}
/**