[MDEV-18931] Rename Mariabackup's xtrabackup_* files to mariadb_backup_* Created: 2019-03-14 Updated: 2023-08-07 Resolved: 2023-03-09 |
|
| Status: | Closed |
| Project: | MariaDB Server |
| Component/s: | mariabackup |
| Fix Version/s: | 11.1.0 |
| Type: | Task | Priority: | Critical |
| Reporter: | Geoff Montee (Inactive) | Assignee: | Alexander Barkov |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | Preview_11.1, mariabackup | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
|
Mariabackup creates a few files of the form xtrabackup_*. e.g.:
Maybe we should rename these to mariabackup_*? |
| Comments |
| Comment by Sergei Golubchik [ 2020-09-10 ] |
|
any compatibility concerns? |
| Comment by Vladislav Lesin [ 2020-09-21 ] |
|
serg We do not guaranty mariabackup compatibility among major versions. Besides, we already did more serious renames from compatibility perspective, like mysqld->mariadbd, mariabackup->mariadb-backup etc. But we used symlinks for that, what is reasonable and can be used for this issue too. |
| Comment by Sergei Golubchik [ 2020-09-21 ] |
|
I don't understand. How do you suggest to use symlinks here? |
| Comment by Vladislav Lesin [ 2020-09-22 ] |
|
We could create mariabackup_* files instead of xtrabackup_*, and create xtrabackup_* symlinks to mariabackup_* to let existing user scripts to use old names. |
| Comment by Sergei Golubchik [ 2020-12-10 ] |
|
why should this be done at all? |
| Comment by Alexander Barkov [ 2023-02-03 ] |
|
As the canonical name is now "mariadb-backup", the files should probably be renamed to mariadb_backup_*. |
| Comment by Alexander Barkov [ 2023-02-03 ] |
|
serg, please review this patch: Thanks! |
| Comment by Alexander Barkov [ 2023-03-07 ] |
|
Hi vlad.lesin, Please review a new patch version: https://github.com/MariaDB/server/commit/39184ac9a0cb2cb56ef286a6b23b3becb729d2ee Thanks. |
| Comment by Vladislav Lesin [ 2023-03-07 ] |
|
bar, I have some doubts about xtrabackup_galera_info renaming. It's used in ./scripts/wsrep_sst_mariabackup.sh for creating state for adding new node to galera cluster. If we just rename it and don't fix the script, it will be broken. I think there should be the same logic in this script as for backup preparing, i.e. try to use old file name, and then the new one. sysprg, could you please suggest how to fix the script? |
| Comment by Vladislav Lesin [ 2023-03-09 ] |
|
Taking into account that the galera sst script for mariabackup will be fixed by Julius, https://github.com/MariaDB/server/commit/0f97bd1244d7c7618698b46327dc2a0900fd8026 looks good to me. |