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
Add max_allowed_packet diagnostic check #13497
Conversation
$status = DiagnosticResult::STATUS_WARNING; | ||
$pretty = MetricsFormatter::getPrettySizeFromBytes($maxPacketBytes['Value'], 'M'); | ||
$configured = str_replace(array(' M', ' M'), 'MB', $pretty); | ||
$comment = Piwik::translate('Diagnostics_MysqlMaxPacketSizeWarning', array('32MB', $configured)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should not be hardcoded I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The referenced issue suggest to set it to at least 64MB, but guess 32MB might be fine as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't remember exactly but we talked about it in chat or so. Somewhere 32 was said. Can change it to 64.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a small comment. Otherwise looks good and worked locally.
$status = DiagnosticResult::STATUS_WARNING; | ||
$pretty = MetricsFormatter::getPrettySizeFromBytes($maxPacketBytes['Value'], 'M'); | ||
$configured = str_replace(array(' M', ' M'), 'MB', $pretty); | ||
$comment = Piwik::translate('Diagnostics_MysqlMaxPacketSizeWarning', array('32MB', $configured)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The referenced issue suggest to set it to at least 64MB, but guess 32MB might be fine as well?
fix #1781
thought about linking to https://matomo.org/faq/troubleshooting/faq_183/ but the FAQ is actually not very helpful.