-
Notifications
You must be signed in to change notification settings - Fork 189
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
Nat minor2 #60
Nat minor2 #60
Conversation
postgresqltuner.pl
Outdated
print_report_ok("huge_pages=on, therefore PostgreSQL needs Huge Pages"); | ||
} | ||
elsif (get_setting('huge_pages') eq 'try') { | ||
print_report_bad("huge_pages=on, therefore PostgreSQL will try to use Huge Pages, if they are enabled"); |
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.
You have try
set as a print_report_bad
level message, while completely disabled is only medium
advice.
If anything on
should be a print_report_warn
, since the server will completely fail to start if it has no available huge pages.
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.
You are right, thank you!
Thanks again ! |
More janitor-related work, added the --nocolor option, various comments