robertmu commented on issue #57: URL: https://github.com/apache/cloudberry-backup/issues/57#issuecomment-3395675407
Good catch! However I think that these lines should be kept in the `Makefile`. The reasoning is tied to our goal of ensuring `gpbackup` supports a wide range of use cases by maintaining backward compatibility with older GPDB versions, alongside our primary support for Cloudberry Database. This flexibility is critical for certain migration scenarios. For example, it becomes necessary in constrained environments where a user can only have one database system running at a time due to resource limitations. In this situation, the migration process is necessarily: 1. Use `gpbackup` to take a full backup of the old database (for instance, GPDB 7). 2. Shut down the old database completely. 3. Install and start the new database (e.g., Cloudberry Database). 4. Use `gprestore` to restore the data from the backup. This workflow means there is a definite requirement for `gpbackup` to reliably back up from older versions like GPDB 7 for a successful restore to Cloudberry Database. The `unit_all_gpdb_versions` target is therefore a valuable tool for our developers. It provides a quick way to run local regression tests to ensure that code changes do not inadvertently break compatibility for these scenarios. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
