db/pg_probackup
1
0
Fork 0
mirror of https://github.com/postgrespro/pg_probackup.git synced 2025-10-31 00:17:52 +02:00
Code Issues Releases Activity

code cleanup

Browse Source
This commit is contained in:
Anastasia
2017-05-02 15:13:32 +03:00
parent 01a21462ac
commit 419008d9de
3 changed files with 23 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
backup.c
View File

@@ -1312,7 +1312,7 @@ add_pgdata_files(parray *files, const char *root)
parray_remove(files, i);
i--;
}
/* compress map file it is not data file */
/* compress map file is not data file */
else if (path_len > 4 &&
strncmp(file->path + (path_len - 4), ".cfm", 4) == 0)
{

22
restore.c
View File

@@ -209,19 +209,24 @@ do_restore_or_validate(time_t target_backup_id,
/*
* Validate backups from base_full_backup to dest_backup.
* And restore if subcommand is RESTORE.
* TODO what if we found out that backup is not valid?
*/
for (i = base_full_backup_index; i >= dest_backup_index; i--)
{
pgBackup *backup = (pgBackup *) parray_get(backups, i);
if (backup->status == BACKUP_STATUS_OK)
{
pgBackupValidate(backup);
}
/* We ensured that all backups are valid, now restore if required */
if (is_restore)
{
for (i = base_full_backup_index; i >= dest_backup_index; i--)
{
pgBackup *backup = (pgBackup *) parray_get(backups, i);
if (backup->status == BACKUP_STATUS_OK)
restore_backup(backup);
else
elog(ERROR, "backup %s is not valid",
base36enc(backup->start_time));
}
}
@@ -548,9 +553,8 @@ create_directory:
* Check that all tablespace mapping entries have correct linked directory
* paths. Linked directories must be empty or do not exist.
*
* If tablespace-mapping option is supplied all OLDDIR entries should have
* If tablespace-mapping option is supplied, all OLDDIR entries must have
* entries in tablespace_map file.
* TODO review
*/
static void
check_tablespace_mapping(pgBackup *backup)
@@ -568,7 +572,7 @@ check_tablespace_mapping(pgBackup *backup)
elog(LOG, "check tablespace directories of backup %s", base36enc(backup->start_time));
/* 1 - OLDDIR should has an entry in links */
/* 1 - each OLDDIR must have an entry in tablespace_map file (links) */
for (cell = tablespace_dirs.head; cell; cell = cell->next)
{
tmp_file->linked = cell->old_dir;
@@ -579,7 +583,7 @@ check_tablespace_mapping(pgBackup *backup)
cell->old_dir);
}
/* 2 - all linked directories should be empty */
/* 2 - all linked directories must be empty */
for (i = 0; i < parray_num(links); i++)
{
pgFile *link = (pgFile *) parray_get(links, i);

16
validate.c
View File

@@ -18,13 +18,11 @@ static void pgBackupValidateFiles(void *arg);
typedef struct
{
parray *files;
bool validate_crc;
bool corrupted;
} validate_files_args;
/*
* Validate backup files.
* TODO review
*/
void
pgBackupValidate(pgBackup *backup)
@@ -63,12 +61,6 @@ pgBackupValidate(pgBackup *backup)
{
validate_files_args *arg = pg_malloc(sizeof(validate_files_args));
arg->files = files;
arg->validate_crc = true;
/* TODO Why didn't we validate checksums on restore before? */
// if (backup_subcmd == RESTORE)
// arg->validate_crc = false;
arg->corrupted = false;
validate_threads_args[i] = arg;
pthread_create(&validate_threads[i], NULL, (void *(*)(void *)) pgBackupValidateFiles, arg);
@@ -99,7 +91,7 @@ pgBackupValidate(pgBackup *backup)
}
/*
* Validate files in the backup with size or CRC.
* Validate files in the backup.
* NOTE: If file is not valid, do not use ERROR log message,
* rather throw a WARNING and set arguments->corrupted = true.
* This is necessary to update backup status.
@@ -109,6 +101,7 @@ pgBackupValidateFiles(void *arg)
{
int i;
validate_files_args *arguments = (validate_files_args *)arg;
pg_crc32 crc;
for (i = 0; i < parray_num(arguments->files); i++)
{
@@ -158,10 +151,6 @@ pgBackupValidateFiles(void *arg)
return;
}
if (arguments->validate_crc)
{
pg_crc32 crc;
crc = pgFileGetCRC(file);
if (crc != file->crc)
{
@@ -172,4 +161,3 @@ pgBackupValidateFiles(void *arg)
}
}
}
}