fix(server): database backups compatible with deduplication

[Scrumplex]

Description

gzip --rsyncable has a slightly worse compression ratio, but allows for efficient deduplication and, as the name implies, faster rsync operations.

How Has This Been Tested?

• npm run test

Checklist:

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation if applicable

[zackpollard]

Had a think about this, in general handling specific scenarios like this isn't something we would want to do, but I think this will help more than just rsync. If someone runs further backups using another tool like restic of these dumps, then this should help the de-duplication algorithm in those tools too.

[akostadinov]

Why not use zstd --rsyncable? Seems like a better and faster compression used by linux kernel nowadays.

[Hoempi]

Had a think about this, in general handling specific scenarios like this isn't something we would want to do, but I think this will help more than just rsync. If someone runs further backups using another tool like restic of these dumps, then this should help the de-duplication algorithm in those tools too.

It if you take this in consideration you possibly should add an option to not zip at all. At least restic‘s built-in combination on deduplication and compression does an even better job. I assume it’s the same for other backup tools as well.

Source: restic/restic#3821

For me this is a reason to disable the built-in function and keep my own pgdump_all script.

And as a fellow developer I would also not add the option to disable the compression to the Immich server, as this is possibly an edge case and the adding of an option most likely will confuse most users.

[akostadinov]

most likely will confuse most users

I don't apprciate such kind of thinking. It's nice to have sensible defaults and smaller UI but there shouldn't be any issues for advanced options button to choose what you need. For theis particular case, probably not worth it though.