[MDEV-31505] Deprecate mariabackup --innobackupex mode Created: 2023-06-20 Updated: 2023-09-26 Resolved: 2023-09-04 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | Backup |
| Fix Version/s: | 11.3.0 |
| Type: | Task | Priority: | Blocker |
| Reporter: | Sergei Golubchik | Assignee: | Alexander Barkov |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | Preview_11.3 | ||
| Issue Links: |
|
||||||||||||||||||||
| Description |
|
Using mariabackup with the --innobackupex switch should print a deprecation warning ER_WARN_DEPRECATED_SYNTAX or ER_WARN_DEPRECATED_SYNTAX_NO_REPLACEMENT. Also, mariadb-backup --innobackupex prints a wrong warning:
It's generated in my_init() which incorrectly thinks that the program name is --innobackupex rather than mariadb-backup. |
| Comments |
| Comment by Alexander Barkov [ 2023-08-31 ] | ||||||
|
Hello serg. Can you please review this patch: https://github.com/MariaDB/server/commit/855ad7484e0c3148c9f0858842aaa60deceb8240 ? Thanks. | ||||||
| Comment by Sergei Golubchik [ 2023-08-31 ] | ||||||
|
code-wise 855ad7484e0c is ok to push. I recommend to clarify the test comment before pushing though | ||||||
| Comment by Timofey Turenko [ 2023-09-21 ] | ||||||
|
it seems something is wrong with command line processing: if '--innobackupex' is at the end of command line mariadb-backup gives `unknown option` error:
if '--innobackupex' is the first argument mariadb-backup gives `Missing argument` error:
tested with https://mirrors.xtom.ee/mariadb//mariadb-11.3.0/bintar-linux-systemd-x86_64/mariadb-11.3.0-preview-linux-systemd-x86_64.tar.gz and Ubuntu Focal |