Skip to content

Commit 265ae91

Browse files
committed
Remove code for sub-par recommendation related to "innodb_redo_log_capacity" setting
The calculation of the "innodb_log_size_pct" value was kept in. Resolves #784
1 parent 83789de commit 265ae91

File tree

1 file changed

+0
-65
lines changed

1 file changed

+0
-65
lines changed

mysqltuner.pl

Lines changed: 0 additions & 65 deletions
Original file line numberDiff line numberDiff line change
@@ -6437,71 +6437,6 @@ sub mysql_innodb {
64376437
}
64386438
}
64396439
}
6440-
if ( $mycalc{'innodb_log_size_pct'} < 20
6441-
or $mycalc{'innodb_log_size_pct'} > 30 )
6442-
{
6443-
if ( defined $myvar{'innodb_redo_log_capacity'} ) {
6444-
badprint
6445-
"Ratio InnoDB redo log capacity / InnoDB Buffer pool size ("
6446-
. $mycalc{'innodb_log_size_pct'} . "%): "
6447-
. hr_bytes( $myvar{'innodb_redo_log_capacity'} ) . " / "
6448-
. hr_bytes( $myvar{'innodb_buffer_pool_size'} )
6449-
. " should be equal to 25%";
6450-
push( @adjvars,
6451-
"innodb_redo_log_capacity should be (="
6452-
. hr_bytes_rnd( $myvar{'innodb_buffer_pool_size'} / 4 )
6453-
. ") if possible, so InnoDB Redo log Capacity equals 25% of buffer pool size."
6454-
);
6455-
push( @generalrec,
6456-
"Be careful, increasing innodb_redo_log_capacity means higher crash recovery mean time"
6457-
);
6458-
}
6459-
else {
6460-
badprint "Ratio InnoDB log file size / InnoDB Buffer pool size ("
6461-
. $mycalc{'innodb_log_size_pct'} . "%): "
6462-
. hr_bytes( $myvar{'innodb_log_file_size'} ) . " * "
6463-
. $myvar{'innodb_log_files_in_group'} . " / "
6464-
. hr_bytes( $myvar{'innodb_buffer_pool_size'} )
6465-
. " should be equal to 25%";
6466-
push(
6467-
@adjvars,
6468-
"innodb_log_file_size should be (="
6469-
. hr_bytes_rnd(
6470-
$myvar{'innodb_buffer_pool_size'} /
6471-
$myvar{'innodb_log_files_in_group'} / 4
6472-
)
6473-
. ") if possible, so InnoDB total log file size equals 25% of buffer pool size."
6474-
);
6475-
push( @generalrec,
6476-
"Be careful, increasing innodb_log_file_size / innodb_log_files_in_group means higher crash recovery mean time"
6477-
);
6478-
}
6479-
if ( mysql_version_le( 5, 6, 2 ) ) {
6480-
push( @generalrec,
6481-
"For MySQL 5.6.2 and lower, total innodb_log_file_size should have a ceiling of (4096MB / log files in group) - 1MB."
6482-
);
6483-
}
6484-
6485-
}
6486-
else {
6487-
if ( defined $myvar{'innodb_redo_log_capacity'} ) {
6488-
goodprint
6489-
"Ratio InnoDB Redo Log Capacity / InnoDB Buffer pool size: "
6490-
. hr_bytes( $myvar{'innodb_redo_log_capacity'} ) . "/"
6491-
. hr_bytes( $myvar{'innodb_buffer_pool_size'} )
6492-
. " should be equal to 25%";
6493-
}
6494-
else {
6495-
push( @generalrec,
6496-
"Before changing innodb_log_file_size and/or innodb_log_files_in_group read this: https://bit.ly/2TcGgtU"
6497-
);
6498-
goodprint "Ratio InnoDB log file size / InnoDB Buffer pool size: "
6499-
. hr_bytes( $myvar{'innodb_log_file_size'} ) . " * "
6500-
. $myvar{'innodb_log_files_in_group'} . "/"
6501-
. hr_bytes( $myvar{'innodb_buffer_pool_size'} )
6502-
. " should be equal to 25%";
6503-
}
6504-
}
65056440

65066441
# InnoDB Buffer Pool Instances (MySQL 5.6.6+)
65076442
if ( not mysql_version_ge( 10, 4 )

0 commit comments

Comments
 (0)