Comment fixes.

Jeff Davis, somewhat edited by me

diff --git a/src/backend/commands/tablecmds.c b/src/backend/commands/tablecmds.c
index 0d3a74ea3720eb38f644515e98535732f248c40d..359d478592b2528211f1556a75900ffd77594a77 100644 (file)
--- a/src/backend/commands/tablecmds.c
+++ b/src/backend/commands/tablecmds.c
@@ -8784,9 +8784,10 @@ copy_relation_data(SMgrRelation src, SMgrRelation dst,
        pfree(buf);
 
        /*
-        * If the rel isn't temp, we must fsync it down to disk before it's safe
-        * to commit the transaction.  (For a temp rel we don't care since the rel
-        * will be uninteresting after a crash anyway.)
+        * If the rel is WAL-logged, must fsync before commit.  We use heap_sync
+        * to ensure that the toast table gets fsync'd too.  (For a temp or
+        * unlogged rel we don't care since the data will be gone after a crash
+        * anyway.)
         *
         * It's obvious that we must do this when not WAL-logging the copy. It's
         * less obvious that we have to do it even if we did WAL-log the copied

diff --git a/src/backend/storage/file/reinit.c b/src/backend/storage/file/reinit.c
index 1912dd8e7aeb76729885475346293151c9a18061..0e2f824a885d5fe0b279b2301baf6b2ca5754af0 100644 (file)
--- a/src/backend/storage/file/reinit.c
+++ b/src/backend/storage/file/reinit.c
@@ -337,7 +337,6 @@ ResetUnloggedRelationsInDbspaceDir(const char *dbspacedirname, int op)
                        copy_file(srcpath, dstpath);
                }
 
-               /* Done with the first pass. */
                FreeDir(dbspace_dir);
        }
 }