Fix bogus comment that claimed that the new BACKUP METHOD line in
authorHeikki Linnakangas <heikki.linnakangas@iki.fi>
Tue, 16 Aug 2011 09:21:50 +0000 (12:21 +0300)
committerHeikki Linnakangas <heikki.linnakangas@iki.fi>
Tue, 16 Aug 2011 09:23:51 +0000 (12:23 +0300)
backup_label was new in 9.0. Spotted by Fujii Masao.

src/backend/access/transam/xlog.c

index 11035e6b4cdcba9f5265352f45d54bb2a0995662..a462288a9ce0a154a7b1dc02038956186f1a93eb 100644 (file)
@@ -9826,9 +9826,9 @@ read_backup_label(XLogRecPtr *checkPointLoc, bool *backupEndRequired)
                (errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE),
                 errmsg("invalid data in file \"%s\"", BACKUP_LABEL_FILE)));
    /*
-    * BACKUP METHOD line is new in 9.0. Don't complain if it doesn't exist,
-    * in case you're restoring from a backup taken with an 9.0 beta version
-    * that didn't emit it.
+    * BACKUP METHOD line is new in 9.2. We can't restore from an older backup
+    * anyway, but since the information on it is not strictly required, don't
+    * error out if it's missing for some reason.
     */
    if (fscanf(lfp, "BACKUP METHOD: %19s", backuptype) == 1)
    {