- there's still no reply in http://bugs.mysql.com/bug.php?id=68635
We obviously cannot afford similar changes for every new statement
type, so we should do something about it. What about adding
performance_schema_max_statement_classes = 167 to
mysql-test/include/default_mysqld.cnf?
- the changes related to digest values is another "gift" from
performance schema. Not sure what to do about it. Probably just
"--replace-column ... <digest>" in the corresponding tests?
- please use OS_FILE_MAX_PATH instead of hard-coded (and
inconsistent) constants in log_group_archive() and
log_group_recover_from_archive_file(). I don't think it then makes
sense to assert on sufficient buffer space, as a lot of things
would break if paths are longer than that. But you can keep the
assertion if you like. The rest of InnoDB code seems to assume that
never happens, but I agree we should fail *if* it does.
- multiple lines violate the 80 chars limit, try:
: bzr diff -c-1 | grep -0 '^+ ' | awk 'length > 82'
- there's still no reply in http:// bugs.mysql. com/bug. php?id= 68635 e_schema_ max_statement_ classes = 167 to test/include/ default_ mysqld. cnf?
We obviously cannot afford similar changes for every new statement
type, so we should do something about it. What about adding
performanc
mysql-
- the changes related to digest values is another "gift" from -replace- column ... <digest>" in the corresponding tests?
performance schema. Not sure what to do about it. Probably just
"-
- please use OS_FILE_MAX_PATH instead of hard-coded (and group_recover_ from_archive_ file(). I don't think it then makes
inconsistent) constants in log_group_archive() and
log_
sense to assert on sufficient buffer space, as a lot of things
would break if paths are longer than that. But you can keep the
assertion if you like. The rest of InnoDB code seems to assume that
never happens, but I agree we should fail *if* it does.