Skip to content

Commit

Permalink
WL#15652: Deprecate mysqlpump
Browse files Browse the repository at this point in the history
Added a deprecation warning
Adjusted the tests to account for it.

Change-Id: I117888a56cd20e7550199f0b0b139c1648fdc436
  • Loading branch information
gkodinov committed Apr 5, 2023
1 parent 84ef6fe commit dd81a0e
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 0 deletions.
1 change: 1 addition & 0 deletions client/dump/program.cc
Original file line number Diff line number Diff line change
Expand Up @@ -271,6 +271,7 @@ const char *load_default_groups[] = {
"mysqlpump", nullptr};

int main(int argc, char **argv) {
CLIENT_WARN_DEPRECATED("mysqlpump", "mysqldump");
Program program;
program.run(argc, argv);
return 0;
Expand Down
1 change: 1 addition & 0 deletions mysql-test/r/mysqlpump.result
Original file line number Diff line number Diff line change
Expand Up @@ -461,6 +461,7 @@ DROP DATABASE db1;
DROP DATABASE db2;
DROP DATABASE db3;
DROP DATABASE db1_1gb;
WARNING: mysqlpump is deprecated and will be removed in a future version. Use mysqldump instead.
SHOW VARIABLES LIKE "master_info_repository";
Variable_name Value
master_info_repository TABLE
Expand Down
2 changes: 2 additions & 0 deletions mysql-test/r/mysqlpump_bugs.result
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,12 @@
# Bug#23102944: MYSQLPUMP LEAKS MEMORY IN CASE OF ERRORS.
#
# testing a failed connection: must not crash
WARNING: mysqlpump is deprecated and will be removed in a future version. Use mysqldump instead.
mysqlpump: Got error: 2005: Unknown MySQL server host 'non_existent_ghost' (errno) while connecting to the MySQL server
mysqlpump: [ERROR] (11)
Dump process encountered error and will not continue.
# testing an invalid option: must not crash
WARNING: mysqlpump is deprecated and will be removed in a future version. Use mysqldump instead.
mysqlpump: [ERROR] unknown variable 'parallelism=3'.
mysqlpump: [ERROR] (7) Error during handling options
Dump process encountered error and will not continue.
Expand Down
1 change: 1 addition & 0 deletions mysql-test/r/ssl_cache.result
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ ERROR: --ssl-session-data specified but the session was not reused.
# try mysqltest
mysqltest: --ssl-session-data specified but the session was not reused.
# try mysqlpump
WARNING: mysqlpump is deprecated and will be removed in a future version. Use mysqldump instead.
--ssl-session-data specified but the session was not reused.
# FR12.1: make sure --ssl-session-data-continue-on-failed-reuse works
# try mysql
Expand Down
1 change: 1 addition & 0 deletions mysql-test/r/ssl_cache_openssl1_0.result
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ ERROR: --ssl-session-data specified but the session was not reused.
# try mysqltest
mysqltest: --ssl-session-data specified but the session was not reused.
# try mysqlpump
WARNING: mysqlpump is deprecated and will be removed in a future version. Use mysqldump instead.
--ssl-session-data specified but the session was not reused.
# FR12.1: make sure --ssl-session-data-continue-on-failed-reuse works
# try mysql
Expand Down
1 change: 1 addition & 0 deletions mysql-test/r/ssl_cache_tls13.result
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ ERROR: --ssl-session-data specified but the session was not reused.
# try mysqltest
mysqltest: --ssl-session-data specified but the session was not reused.
# try mysqlpump
WARNING: mysqlpump is deprecated and will be removed in a future version. Use mysqldump instead.
--ssl-session-data specified but the session was not reused.
# FR12.1: make sure --ssl-session-data-continue-on-failed-reuse works
# try mysql
Expand Down

0 comments on commit dd81a0e

Please sign in to comment.