[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:
PartOf
is part of MDEV-30201 Remove MySQL names Open
Relates
relates to MDEV-14746 mariabackup doesn't read [mariabackup... Closed
relates to MDEV-18347 mariabackup doesn't read all server o... Closed
relates to MDEV-18405 Add Mariabackup option to set gtid_sl... Stalled
relates to MDEV-18917 Don't create xtrabackup_binlog_pos_in... Closed
relates to MDEV-19246 Change database and table used for Ma... Closed
relates to MDEV-18985 Remove support for XtraDB's changed p... Confirmed
relates to MDEV-19264 Better support MariaDB GTID for Maria... Closed
relates to MDEV-20552 Remove innobackupex mode from Mariaba... Stalled
relates to MDEV-20556 Remove references to "xtrabackup" and... Closed
relates to MDEV-30822 Adjust wsrep_sst_mariabackup.sh to un... Open

 Description   

Mariabackup creates a few files of the form xtrabackup_*. e.g.:

$ ls -l backup/xtrabackup_*
-rw-r----- 1 root root  38 Mar 13 18:43 backup/xtrabackup_binlog_info
-rw-r--r-- 1 root root  25 Mar 13 18:44 backup/xtrabackup_binlog_pos_innodb
-rw-r----- 1 root root  99 Mar 13 18:44 backup/xtrabackup_checkpoints
-rw-r----- 1 root root 572 Mar 13 18:43 backup/xtrabackup_info

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:
https://github.com/MariaDB/server/commit/ba70039c89b91dc1017d1451864bcd3caa3cbda0

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.

Generated at Thu Feb 08 08:47:49 UTC 2024 using Jira 8.20.16#820016-sha1:9d11dbea5f4be3d4cc21f03a88dd11d8c8687422.