diff --git a/src/applications/config/check/PhabricatorSetupCheckMySQL.php b/src/applications/config/check/PhabricatorSetupCheckMySQL.php index aec1b6ab23..9ea3ca6f6d 100644 --- a/src/applications/config/check/PhabricatorSetupCheckMySQL.php +++ b/src/applications/config/check/PhabricatorSetupCheckMySQL.php @@ -1,241 +1,281 @@ establishConnection('w'); try { $value = queryfx_one($conn_raw, 'SELECT @@%Q', $key); $value = $value['@@'.$key]; } catch (AphrontQueryException $ex) { $value = null; } return $value; } protected function executeChecks() { $max_allowed_packet = self::loadRawConfigValue('max_allowed_packet'); $recommended_minimum = 1024 * 1024; if ($max_allowed_packet < $recommended_minimum) { $message = pht( "MySQL is configured with a very small 'max_allowed_packet' (%d), ". "which may cause some large writes to fail. Strongly consider raising ". "this to at least %d in your MySQL configuration.", $max_allowed_packet, $recommended_minimum); $this->newIssue('mysql.max_allowed_packet') ->setName(pht('Small MySQL "max_allowed_packet"')) ->setMessage($message) ->addMySQLConfig('max_allowed_packet'); } $modes = self::loadRawConfigValue('sql_mode'); $modes = explode(',', $modes); + if (!in_array('STRICT_ALL_TABLES', $modes)) { $summary = pht( 'MySQL is not in strict mode, but using strict mode is strongly '. 'encouraged.'); $message = pht( "On your MySQL instance, the global %s is not set to %s. ". "It is strongly encouraged that you enable this mode when running ". "Phabricator.\n\n". "By default MySQL will silently ignore some types of errors, which ". "can cause data loss and raise security concerns. Enabling strict ". "mode makes MySQL raise an explicit error instead, and prevents this ". "entire class of problems from doing any damage.\n\n". "You can find more information about this mode (and how to configure ". "it) in the MySQL manual. Usually, it is sufficient to add this to ". "your %s file (in the %s section) and then restart %s:\n\n". "%s\n". "(Note that if you run other applications against the same database, ". "they may not work in strict mode. Be careful about enabling it in ". "these cases.)", phutil_tag('tt', array(), 'sql_mode'), phutil_tag('tt', array(), 'STRICT_ALL_TABLES'), phutil_tag('tt', array(), 'my.cnf'), phutil_tag('tt', array(), '[mysqld]'), phutil_tag('tt', array(), 'mysqld'), phutil_tag('pre', array(), 'sql_mode=STRICT_ALL_TABLES')); $this->newIssue('mysql.mode') ->setName(pht('MySQL STRICT_ALL_TABLES Mode Not Set')) ->setSummary($summary) ->setMessage($message) ->addMySQLConfig('sql_mode'); } + if (in_array('ONLY_FULL_GROUP_BY', $modes)) { + $summary = pht( + 'MySQL is in ONLY_FULL_GROUP_BY mode, but using this mode is strongly '. + 'discouraged.'); + + $message = pht( + "On your MySQL instance, the global %s is set to %s. ". + "It is strongly encouraged that you disable this mode when running ". + "Phabricator.\n\n". + "With %s enabled, MySQL rejects queries for which the select list ". + "or (as of MySQL 5.0.23) %s list refer to nonaggregated columns ". + "that are not named in the %s clause. More importantly, Phabricator ". + "does not work properly with this mode enabled.\n\n". + "You can find more information about this mode (and how to configure ". + "it) in the MySQL manual. Usually, it is sufficient to change the %s ". + "in your %s file (in the %s section) and then restart %s:\n\n". + "%s\n". + "(Note that if you run other applications against the same database, ". + "they may not work with %s. Be careful about enabling ". + "it in these cases and consider migrating Phabricator to a different ". + "database.)", + phutil_tag('tt', array(), 'sql_mode'), + phutil_tag('tt', array(), 'ONLY_FULL_GROUP_BY'), + phutil_tag('tt', array(), 'ONLY_FULL_GROUP_BY'), + phutil_tag('tt', array(), 'HAVING'), + phutil_tag('tt', array(), 'GROUP BY'), + phutil_tag('tt', array(), 'sql_mode'), + phutil_tag('tt', array(), 'my.cnf'), + phutil_tag('tt', array(), '[mysqld]'), + phutil_tag('tt', array(), 'mysqld'), + phutil_tag('pre', array(), 'sql_mode=STRICT_ALL_TABLES'), + phutil_tag('tt', array(), 'ONLY_FULL_GROUP_BY')); + + $this->newIssue('mysql.mode') + ->setName(pht('MySQL ONLY_FULL_GROUP_BY Mode Set')) + ->setSummary($summary) + ->setMessage($message) + ->addMySQLConfig('sql_mode'); + } $stopword_file = self::loadRawConfigValue('ft_stopword_file'); if (!PhabricatorDefaultSearchEngineSelector::shouldUseElasticSearch()) { if ($stopword_file === null) { $summary = pht( 'Your version of MySQL does not support configuration of a '. 'stopword file. You will not be able to find search results for '. 'common words.'); $message = pht( "Your MySQL instance does not support the %s option. You will not ". "be able to find search results for common words. You can gain ". "access to this option by upgrading MySQL to a more recent ". "version.\n\n". "You can ignore this warning if you plan to configure ElasticSearch ". "later, or aren't concerned about searching for common words.", phutil_tag('tt', array(), 'ft_stopword_file')); $this->newIssue('mysql.ft_stopword_file') ->setName(pht('MySQL ft_stopword_file Not Supported')) ->setSummary($summary) ->setMessage($message) ->addMySQLConfig('ft_stopword_file'); } else if ($stopword_file == '(built-in)') { $root = dirname(phutil_get_library_root('phabricator')); $stopword_path = $root.'/resources/sql/stopwords.txt'; $stopword_path = Filesystem::resolvePath($stopword_path); $namespace = PhabricatorEnv::getEnvConfig('storage.default-namespace'); $summary = pht( 'MySQL is using a default stopword file, which will prevent '. 'searching for many common words.'); $message = pht( "Your MySQL instance is using the builtin stopword file for ". "building search indexes. This can make Phabricator's search ". "feature less useful.\n\n". "Stopwords are common words which are not indexed and thus can not ". "be searched for. The default stopword file has about 500 words, ". "including various words which you are likely to wish to search ". "for, such as 'various', 'likely', 'wish', and 'zero'.\n\n". "To make search more useful, you can use an alternate stopword ". "file with fewer words. Alternatively, if you aren't concerned ". "about searching for common words, you can ignore this warning. ". "If you later plan to configure ElasticSearch, you can also ignore ". "this warning: this stopword file only affects MySQL fulltext ". "indexes.\n\n". "To choose a different stopword file, add this to your %s file ". "(in the %s section) and then restart %s:\n\n". "%s\n". "(You can also use a different file if you prefer. The file ". "suggested above has about 50 of the most common English words.)\n\n". "Finally, run this command to rebuild indexes using the new ". "rules:\n\n". "%s", phutil_tag('tt', array(), 'my.cnf'), phutil_tag('tt', array(), '[mysqld]'), phutil_tag('tt', array(), 'mysqld'), phutil_tag('pre', array(), 'ft_stopword_file='.$stopword_path), phutil_tag( 'pre', array(), "mysql> REPAIR TABLE {$namespace}_search.search_documentfield;")); $this->newIssue('mysql.ft_stopword_file') ->setName(pht('MySQL is Using Default Stopword File')) ->setSummary($summary) ->setMessage($message) ->addMySQLConfig('ft_stopword_file'); } } $min_len = self::loadRawConfigValue('ft_min_word_len'); if ($min_len >= 4) { if (!PhabricatorDefaultSearchEngineSelector::shouldUseElasticSearch()) { $namespace = PhabricatorEnv::getEnvConfig('storage.default-namespace'); $summary = pht( 'MySQL is configured to only index words with at least %d '. 'characters.', $min_len); $message = pht( "Your MySQL instance is configured to use the default minimum word ". "length when building search indexes, which is 4. This means words ". "which are only 3 characters long will not be indexed and can not ". "be searched for.\n\n". "For example, you will not be able to find search results for words ". "like 'SMS', 'web', or 'DOS'.\n\n". "You can change this setting to 3 to allow these words to be ". "indexed. Alternatively, you can ignore this warning if you are ". "not concerned about searching for 3-letter words. If you later ". "plan to configure ElasticSearch, you can also ignore this warning: ". "only MySQL fulltext search is affected.\n\n". "To reduce the minimum word length to 3, add this to your %s file ". "(in the %s section) and then restart %s:\n\n". "%s\n". "Finally, run this command to rebuild indexes using the new ". "rules:\n\n". "%s", phutil_tag('tt', array(), 'my.cnf'), phutil_tag('tt', array(), '[mysqld]'), phutil_tag('tt', array(), 'mysqld'), phutil_tag('pre', array(), 'ft_min_word_len=3'), phutil_tag( 'pre', array(), "mysql> REPAIR TABLE {$namespace}_search.search_documentfield;")); $this->newIssue('mysql.ft_min_word_len') ->setName(pht('MySQL is Using Default Minimum Word Length')) ->setSummary($summary) ->setMessage($message) ->addMySQLConfig('ft_min_word_len'); } } $innodb_pool = self::loadRawConfigValue('innodb_buffer_pool_size'); $innodb_bytes = phutil_parse_bytes($innodb_pool); $innodb_readable = phutil_format_bytes($innodb_bytes); // This is arbitrary and just trying to detect values that the user // probably didn't set themselves. The Mac OS X default is 128MB and // 40% of an AWS EC2 Micro instance is 245MB, so keeping it somewhere // between those two values seems like a reasonable approximation. $minimum_readable = '225MB'; $minimum_bytes = phutil_parse_bytes($minimum_readable); if ($innodb_bytes < $minimum_bytes) { $summary = pht( 'MySQL is configured with a very small innodb_buffer_pool_size, '. 'which may impact performance.'); $message = pht( "Your MySQL instance is configured with a very small %s (%s). ". "This may cause poor database performance and lock exhaustion.\n\n". "There are no hard-and-fast rules to setting an appropriate value, ". "but a reasonable starting point for a standard install is something ". "like 40%% of the total memory on the machine. For example, if you ". "have 4GB of RAM on the machine you have installed Phabricator on, ". "you might set this value to %s.\n\n". "You can read more about this option in the MySQL documentation to ". "help you make a decision about how to configure it for your use ". "case. There are no concerns specific to Phabricator which make it ". "different from normal workloads with respect to this setting.\n\n". "To adjust the setting, add something like this to your %s file (in ". "the %s section), replacing %s with an appropriate value for your ". "host and use case. Then restart %s:\n\n". "%s\n". "If you're satisfied with the current setting, you can safely ". "ignore this setup warning.", phutil_tag('tt', array(), 'innodb_buffer_pool_size'), phutil_tag('tt', array(), $innodb_readable), phutil_tag('tt', array(), '1600M'), phutil_tag('tt', array(), 'my.cnf'), phutil_tag('tt', array(), '[mysqld]'), phutil_tag('tt', array(), '1600M'), phutil_tag('tt', array(), 'mysqld'), phutil_tag('pre', array(), 'innodb_buffer_pool_size=1600M')); $this->newIssue('mysql.innodb_buffer_pool_size') ->setName(pht('MySQL May Run Slowly')) ->setSummary($summary) ->setMessage($message) ->addMySQLConfig('innodb_buffer_pool_size'); } } }