Skip to content

Commit 8e70523

Browse files
author
Marek Młynarski
committed
BUG#35717411 Shell 8.1.1 upgradechecker util uses 8.0 version in generated report
Commit changes instances of MySQL 8.0 version in upgrade checker report to match current MySQL version. Change-Id: I1f5e0a991d89e43442f98c19522f350d4eaaba7b
1 parent 67ca56e commit 8e70523

File tree

2 files changed

+50
-46
lines changed

2 files changed

+50
-46
lines changed

modules/util/upgrade_check.cc

Lines changed: 28 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -463,12 +463,12 @@ class Routine_syntax_check : public Upgrade_check {
463463

464464
protected:
465465
const char *get_title_internal() const override {
466-
return "MySQL 8.0 syntax check for routine-like objects";
466+
return "MySQL syntax check for routine-like objects";
467467
}
468468

469469
const char *get_description_internal() const override {
470470
return "The following objects did not pass a syntax check with the latest "
471-
"MySQL 8.0 grammar. A common reason is that they reference names "
471+
"MySQL grammar. A common reason is that they reference names "
472472
"that conflict with new reserved keywords. You must update these "
473473
"routine definitions and `quote` any such references before "
474474
"upgrading.";
@@ -564,10 +564,11 @@ bool UNUSED_VARIABLE(register_utf8mb3) = Upgrade_check::register_check(
564564
std::unique_ptr<Sql_upgrade_check> Sql_upgrade_check::get_mysql_schema_check() {
565565
return std::make_unique<Sql_upgrade_check>(
566566
"mysqlSchemaCheck",
567-
"Table names in the mysql schema conflicting with new tables in 8.0",
567+
"Table names in the mysql schema conflicting with new tables in the "
568+
"latest MySQL.",
568569
std::vector<std::string>{
569-
"SELECT TABLE_SCHEMA, TABLE_NAME, 'Table name used in mysql schema "
570-
"in 8.0' as WARNING FROM INFORMATION_SCHEMA.TABLES WHERE "
570+
"SELECT TABLE_SCHEMA, TABLE_NAME, 'Table name used in mysql schema.' "
571+
"as WARNING FROM INFORMATION_SCHEMA.TABLES WHERE "
571572
"LOWER(TABLE_SCHEMA) = 'mysql' and LOWER(TABLE_NAME) IN "
572573
"('catalogs', 'character_sets', 'collations', 'column_type_elements',"
573574
" 'columns', 'dd_properties', 'events', 'foreign_key_column_usage', "
@@ -581,9 +582,9 @@ std::unique_ptr<Sql_upgrade_check> Sql_upgrade_check::get_mysql_schema_check() {
581582
"'innodb_dynamic_metadata', 'password_history', 'role_edges');"},
582583
Upgrade_issue::ERROR,
583584
"The following tables in mysql schema have names that will conflict with "
584-
"the ones introduced in 8.0 version. They must be renamed or removed "
585-
"before upgrading (use RENAME TABLE command). This may also entail "
586-
"changes to applications that use the affected tables.");
585+
"the ones introduced in the latest version. They must be renamed or "
586+
"removed before upgrading (use RENAME TABLE command). This may also "
587+
"entail changes to applications that use the affected tables.");
587588
}
588589

589590
namespace {
@@ -624,7 +625,7 @@ std::unique_ptr<Sql_upgrade_check> Sql_upgrade_check::get_zerofill_check() {
624625
Upgrade_issue::NOTICE,
625626
"The following table columns specify a ZEROFILL/display length "
626627
"attributes. "
627-
"Please be aware that they will be ignored in MySQL 8.0");
628+
"Please be aware that they will be ignored in the latest MySQL version.");
628629
}
629630

630631
// Zerofill is still available in 8.0.11
@@ -644,13 +645,14 @@ Sql_upgrade_check::get_nonnative_partitioning_check() {
644645
"create_options like '%partitioned%' and upper(engine) not in "
645646
"('INNODB', 'NDB', 'NDBCLUSTER');"},
646647
Upgrade_issue::ERROR,
647-
"In MySQL 8.0 storage engine is responsible for providing its own "
648+
"In the latest MySQL storage engine is responsible for providing its own "
648649
"partitioning handler, and the MySQL server no longer provides generic "
649650
"partitioning support. InnoDB and NDB are the only storage engines that "
650-
"provide a native partitioning handler that is supported in MySQL 8.0. A "
651-
"partitioned table using any other storage engine must be altered—either "
652-
"to convert it to InnoDB or NDB, or to remove its partitioning—before "
653-
"upgrading the server, else it cannot be used afterwards.");
651+
"provide a native partitioning handler that is supported in the latest "
652+
"MySQL. A partitioned table using any other storage engine must be "
653+
"altered—either to convert it to InnoDB or NDB, or to remove its "
654+
"partitioning—before upgrading the server, else it cannot be used "
655+
"afterwards.");
654656
}
655657

656658
namespace {
@@ -704,12 +706,12 @@ Sql_upgrade_check::get_maxdb_sql_mode_flags_check() {
704706
"'sql_mode' and find_in_set('MAXDB', variable_value);"},
705707
Upgrade_issue::WARNING,
706708
"The following DB objects have the obsolete MAXDB option persisted for "
707-
"sql_mode, which will be cleared during upgrade to 8.0. It "
709+
"sql_mode, which will be cleared during the upgrade. It "
708710
"can potentially change the datatype DATETIME into TIMESTAMP if it is "
709711
"used inside object's definition, and this in turn can change the "
710712
"behavior in case of dates earlier than 1970 or later than 2037. If this "
711713
"is a concern, please redefine these objects so that they do not rely on "
712-
"the MAXDB flag before running the upgrade to 8.0.");
714+
"the MAXDB flag before running the upgrade.");
713715
}
714716

715717
namespace {
@@ -752,7 +754,7 @@ Sql_upgrade_check::get_obsolete_sql_mode_flags_check() {
752754
"sqlModeFlagCheck", "Usage of obsolete sql_mode flags",
753755
std::move(queries), Upgrade_issue::NOTICE,
754756
"The following DB objects have obsolete options persisted for sql_mode, "
755-
"which will be cleared during upgrade to 8.0.");
757+
"which will be cleared during the upgrade.");
756758
}
757759

758760
namespace {
@@ -844,8 +846,8 @@ Sql_upgrade_check::get_partitioned_tables_in_shared_tablespaces_check(
844846
"itb.space and it.name like '%#P#%' and it.space_type != "
845847
"'Single';"},
846848
Upgrade_issue::ERROR,
847-
"The following tables have partitions in shared tablespaces. Before "
848-
"upgrading to 8.0 they need to be moved to file-per-table tablespace. "
849+
"The following tables have partitions in shared tablespaces. They need "
850+
"to be moved to file-per-table tablespace before upgrading. "
849851
"You can do this by running query like 'ALTER TABLE table_name "
850852
"REORGANIZE PARTITION X INTO (PARTITION X VALUES LESS THAN (30) "
851853
"TABLESPACE=innodb_file_per_table);'");
@@ -988,8 +990,8 @@ class Removed_functions_check : public Sql_upgrade_check {
988990
"('performance_schema','information_schema','sys','mysql')"},
989991
Upgrade_issue::ERROR,
990992
"Following DB objects make use of functions that have "
991-
"been removed in version 8.0. Please make sure to update them to "
992-
"use supported alternatives before upgrade.") {}
993+
"been removed int the latest MySQL version. Please make sure to "
994+
"update them to use supported alternatives before upgrade.") {}
993995

994996
protected:
995997
Upgrade_issue parse_row(const mysqlshdk::db::IRow *row) override {
@@ -1479,10 +1481,11 @@ Sql_upgrade_check::get_fts_in_tablename_check(const Upgrade_info &info) {
14791481
"from information_schema.tables where table_name like binary "
14801482
"'%FTS%';"},
14811483
Upgrade_issue::ERROR,
1482-
"Upgrading from 5.7 to 8.0 does not support tables with name containing "
1483-
"'FTS' character string. The workaround is to rename the table for the "
1484-
"upgrade - e.g. it is enough to change any letter of the 'FTS' part to a "
1485-
"lower case. It can be renamed back again after the upgrade.");
1484+
"Upgrading from 5.7 to the latest MySQL version does not support tables "
1485+
"with name containing 'FTS' character string. The workaround is to "
1486+
"rename the table for the upgrade - e.g. it is enough to change any "
1487+
"letter of the 'FTS' part to a lower case. It can be renamed back again "
1488+
"after the upgrade.");
14861489
}
14871490

14881491
namespace {

res/upgrade_checker/upgrade_checker.msg

Lines changed: 22 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -45,12 +45,12 @@
4545
https://dev.mysql.com/blog-archive/mysql-8-0-removing-support-for-old-temporal-datatypes/
4646

4747
* routinesSyntaxCheck.title
48-
# MySQL 8.0 syntax check for routine-like objects
48+
# MySQL syntax check for routine-like objects
4949

5050
* routinesSyntaxCheck.description
51-
# The following objects did not pass a syntax check with the latest MySQL 8.0 grammar. A common
52-
# reason is that they reference names that conflict with new reserved keywords. You must update
53-
# these routine definitions and `quote` any such references before upgrading.
51+
# The following objects did not pass a syntax check with the latest MySQL grammar. A common reason
52+
# is that they reference names that conflict with new reserved keywords. You must update these
53+
# routine definitions and `quote` any such references before upgrading.
5454

5555
* routinesSyntaxCheck.docLink
5656
https://dev.mysql.com/doc/refman/en/keywords.html
@@ -77,12 +77,12 @@ https://dev.mysql.com/doc/refman/en/keywords.html
7777
https://dev.mysql.com/doc/refman/8.0/en/charset-unicode-utf8mb3.html
7878

7979
* mysqlSchemaCheck.title
80-
# Table names in the mysql schema conflicting with new tables in 8.0
80+
# Table names in the mysql schema conflicting with new tables in latest MySQL.
8181

8282
* mysqlSchemaCheck.description
8383
# Error: The following tables in mysql schema have names that will conflict with the ones
84-
# introduced in 8.0 version. They must be renamed or removed before upgrading (use RENAME TABLE
85-
# command). This may also entail changes to applications that use the affected tables.
84+
# introduced in the latest version. They must be renamed or removed before upgrading (use RENAME
85+
# TABLE command). This may also entail changes to applications that use the affected tables.
8686

8787
* mysqlSchemaCheck.docLink
8888
https://dev.mysql.com/doc/refman/8.0/en/upgrading-strategies.html#upgrade-prerequisites
@@ -91,9 +91,9 @@ https://dev.mysql.com/doc/refman/8.0/en/upgrading-strategies.html#upgrade-prereq
9191
# Partitioned tables using engines with non native partitioning
9292

9393
* nonNativePartitioningCheck.description
94-
# Error: In MySQL 8.0 storage engine is responsible for providing its own partitioning handler, and
95-
# the MySQL server no longer provides generic partitioning support. InnoDB and NDB are the only
96-
# storage engines that provide a native partitioning handler that is supported in MySQL 8.0. A
94+
# Error: In latest MySQL storage engine is responsible for providing its own partitioning handler,
95+
# and the MySQL server no longer provides generic partitioning support. InnoDB and NDB are the only
96+
# storage engines that provide a native partitioning handler that is supported in latest MySQL. A
9797
# partitioned table using any other storage engine must be altered—either to convert it to InnoDB
9898
# or NDB, or to remove its partitioning—before upgrading the server, else it cannot be used
9999
# afterwards.
@@ -116,10 +116,10 @@ https://dev.mysql.com/doc/refman/8.0/en/upgrading-strategies.html#upgrade-prereq
116116

117117
* maxdbFlagCheck.description
118118
# Warning: The following DB objects have the obsolete MAXDB option persisted for sql_mode, which
119-
# will be cleared during upgrade to 8.0. It can potentially change the datatype DATETIME into
119+
# will be cleared during the upgrade. It can potentially change the datatype DATETIME into
120120
# TIMESTAMP if it is used inside object's definition, and this in turn can change the behavior in
121121
# case of dates earlier than 1970 or later than 2037. If this is a concern, please redefine these
122-
# objects so that they do not rely on the MAXDB flag before running the upgrade to 8.0.
122+
# objects so that they do not rely on the MAXDB flag before running the upgrade.
123123

124124
* maxdbFlagCheck.docLink
125125
https://dev.mysql.com/doc/refman/8.0/en/mysql-nutshell.html#mysql-nutshell-removals
@@ -129,7 +129,7 @@ https://dev.mysql.com/doc/refman/8.0/en/mysql-nutshell.html#mysql-nutshell-remov
129129

130130
* sqlModeFlagCheck.description
131131
# Notice: The following DB objects have obsolete options persisted for sql_mode, which will be
132-
# cleared during upgrade to 8.0.
132+
# cleared during upgrade to 8.1.
133133

134134
* sqlModeFlagCheck.docLink
135135
https://dev.mysql.com/doc/refman/8.0/en/mysql-nutshell.html#mysql-nutshell-removals
@@ -149,8 +149,8 @@ https://dev.mysql.com/doc/refman/8.0/en/string-type-overview.html
149149
# Usage of partitioned tables in shared tablespaces
150150

151151
* partitionedTablesInSharedTablespaceCheck.description
152-
# Error: The following tables have partitions in shared tablespaces. Before upgrading to 8.0 they
153-
# need to be moved to file-per-table tablespace. You can do this by running query like 'ALTER TABLE
152+
# Error: The following tables have partitions in shared tablespaces. They need to be moved to
153+
# file-per-table tablespace before upgrading. You can do this by running query like 'ALTER TABLE
154154
# table_name REORGANIZE PARTITION X INTO (PARTITION X VALUES LESS THAN (30)
155155
# TABLESPACE=innodb_file_per_table);'
156156

@@ -174,8 +174,8 @@ https://dev.mysql.com/doc/refman/8.0/en/upgrading-from-previous-series.html#upgr
174174
# Usage of removed functions
175175

176176
* removedFunctionsCheck.description
177-
# Error: Following DB objects make use of functions that have been removed in version 8.0. Please
178-
# make sure to update them to use supported alternatives before upgrade.
177+
# Error: Following DB objects make use of functions that have been removed int the latest MySQL
178+
# version. Please make sure to update them to use supported alternatives before upgrade.
179179

180180
* removedFunctionsCheck.docLink
181181
https://dev.mysql.com/doc/refman/8.0/en/mysql-nutshell.html#mysql-nutshell-removals
@@ -252,9 +252,10 @@ https://lefred.be/content/mysql-8-0-and-wrong-dates/
252252
# Table names containing 'FTS'
253253

254254
* ftsTablenameCheck.description
255-
# Error: Upgrading from 5.7 to 8.0 does not support tables with name containing 'FTS' character
256-
# string. The workaround is to rename the table for the upgrade - e.g. it is enough to change any
257-
# letter of the 'FTS' part to a lower case. It can be renamed back again after the upgrade.
255+
# Error: Upgrading from 5.7 to latest MySQL version does not support tables with name containing
256+
# 'FTS' character string. The workaround is to rename the table for the upgrade - e.g. it is enough
257+
# to change any letter of the 'FTS' part to a lower case. It can be renamed back again after the
258+
# upgrade.
258259

259260
* ftsTablenameCheck.docLink
260261

@@ -445,7 +446,7 @@ https://dev.mysql.com/doc/refman/8.0/en/mysql-nutshell.html#mysql-nutshell-remov
445446
# Check for deprecated or invalid user authentication methods.
446447

447448
* mysqlDeprecatedAuthMethodCheck.description
448-
# The following user have problems with their assigned authentication method:\n
449+
# The following users are using a deprecated authentication method:\n
449450

450451
* mysqlDeprecatedAuthMethodCheck.docLink
451452

0 commit comments

Comments
 (0)