search

Tokutek / mysql-5.5

TokuDB for MySQL 5.5
http://www.tokutek.com/products/tokudb-for-mysql/
GNU General Public License v2.0
110 stars 38 forks source link

check main.mysql_upgrade failed #16

Closed prohaska closed 10 years ago

prohaska commented 10 years ago 
main.mysql_upgrade                       w6 [ pass ]   3802

MTR's internal check of the test case 'main.mysql_upgrade' failed.
This means that the test case does not preserve the state that existed
before the test case was executed.  Most likely the test case did not
do a proper clean-up. It could also be caused by the previous test run
by this thread, if the server wasn't restarted.
This is the diff of the states of the servers before and after the
test case was executed:
mysqltest: Logging to '/data/tokubuild/mysqls/mysql-5.5.35-tokudb-1391195291-debug-e-linux-x86_64/mysql-test/var/6/tmp/check-mysqld_1.log'.
mysqltest: Results saved in '/data/tokubuild/mysqls/mysql-5.5.35-tokudb-1391195291-debug-e-linux-x86_64/mysql-test/var/6/tmp/check-mysqld_1.result'.
mysqltest: Connecting to server localhost:13060 (socket /tmp/APJlXmV5HD/6/mysqld.1.sock) as 'root', connection 'default', attempt 0 ...
mysqltest: ... Connected.
mysqltest: Start processing test commands from './include/check-testcase.test' ...
mysqltest: ... Done processing test commands.
--- /data/tokubuild/mysqls/mysql-5.5.35-tokudb-1391195291-debug-e-linux-x86_64/mysql-test/var/6/tmp/check-mysqld_1.result       2014-02-01 04:50:13.855271278 +0300
+++ /data/tokubuild/mysqls/mysql-5.5.35-tokudb-1391195291-debug-e-linux-x86_64/mysql-test/var/6/tmp/check-mysqld_1.reject       2014-02-01 04:50:17.821244567 +0300
@@ -320,7 +320,7 @@
 SQL_LOG_OFF    OFF
 SQL_LOW_PRIORITY_UPDATES       OFF
 SQL_MAX_JOIN_SIZE      18446744073709551615
-SQL_MODE
+SQL_MODE       NO_ENGINE_SUBSTITUTION
 SQL_NOTES      ON
 SQL_QUOTE_SHOW_CREATE  ON
 SQL_SAFE_UPDATES       OFF
prohaska commented 10 years ago

about 59 tests fail with sql_mode=NO_ENGINE_SUBSTITUTION. mysql_upgrade passes with sql_mode=NO_ENGINE_SUBSTITUTION. either we fix the 59 failing tests, or we ignore this issue. i vote for the latter.