revert prior patch to fsync directories until portability problems exposed by build...
authorGreg Stark <[email protected]>
Wed, 17 Feb 2010 11:35:51 +0000 (11:35 +0000)
committerGreg Stark <[email protected]>
Wed, 17 Feb 2010 11:35:51 +0000 (11:35 +0000)
src/port/copydir.c

index bd2526918128b84e0636b6ab68bd41ecdab3b5d6..067f2bc91035555f54a361b4d8f7fe89402fe4c8 100644 (file)
@@ -11,7 +11,7 @@
  * as a service.
  *
  * IDENTIFICATION
- *   $PostgreSQL: pgsql/src/port/copydir.c,v 1.16.2.2 2010/02/14 17:50:34 stark Exp $
+ *   $PostgreSQL: pgsql/src/port/copydir.c,v 1.16.2.3 2010/02/17 11:35:51 stark Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -50,7 +50,6 @@ copydir(char *fromdir, char *todir, bool recurse)
 {
    DIR        *xldir;
    struct dirent *xlde;
-   int         dirfd;
    char        fromfile[MAXPGPATH];
    char        tofile[MAXPGPATH];
 
@@ -92,26 +91,6 @@ copydir(char *fromdir, char *todir, bool recurse)
    }
 
    FreeDir(xldir);
-
-   /*
-    * fsync the directory to make sure not just the data but also the
-    * new directory file entries have reached the disk. While needed
-    * by most filesystems, the window got bigger with newer ones like
-    * ext4.
-    */
-   dirfd = BasicOpenFile(todir,
-                         O_RDONLY | PG_BINARY,
-                         S_IRUSR | S_IWUSR);
-   if(dirfd == -1)
-       ereport(ERROR,
-               (errcode_for_file_access(),
-                errmsg("could not open directory for fsync \"%s\": %m", todir)));
-
-   if(pg_fsync(dirfd) == -1)
-       ereport(ERROR,
-               (errcode_for_file_access(),
-                errmsg("could not fsync directory \"%s\": %m", todir)));
-   close(dirfd);
 }
 
 /*