Changeset 2085

Timestamp:

31/01/2008 23:57:14 (4 years ago)

Author:

chris

Message:

Always add file size to timestamp in release mode, to help caching work
better on machines which claim to support high-res timestamps but actually
don't (e.g. Linux, MacOS X).

Never do so in debug mode, to help catch problems with the tests failing
to sleep long enough between operations on different connections that
can break caching subtly and randomly.

File:

• box/trunk/lib/raidfile/RaidFileUtil.cpp (modified) (5 diffs)

box/trunk/lib/raidfile/RaidFileUtil.cpp

@@ -23 +23 @@
 //
 // Function
-//              Name:    RaidFileUtil::RaidFileExists(RaidFileDiscSet &, const std::string &)
-//              Purpose: Check to see the state of a RaidFile on disc (doesn't look at contents,
-//                               just at existense of files)
+//              Name:    RaidFileUtil::RaidFileExists(RaidFileDiscSet &,
+//                       const std::string &, int *, int *, int64_t *)
+//              Purpose: Check to see the state of a RaidFile on disc
+//                       (doesn't look at contents, just at existence of
+//                       files)
 //              Created: 2003/07/11
 //
@@ -55 +57 @@
                         {
                                 (*pRevisionID) = FileModificationTime(st);
-#ifndef HAVE_STRUCT_STAT_ST_MTIMESPEC
-                                // On linux, the time resolution is very low for modification times.
-                                // So add the size to it to give a bit more chance of it changing.
+#ifdef NDEBUG
+                                // The resolution of timestamps may be very
+                                // low, e.g. 1 second. So add the size to it
+                                // to give a bit more chance of it changing.
                                 // TODO: Make this better.
+                                // Disabled in debug mode, to simulate
+                                // filesystem with 1-second timestamp
+                                // resolution, e.g. MacOS X HFS, Linux.
                                 (*pRevisionID) += st.st_size;
 #endif
@@ -72 +78 @@
         int setSize = rDiscSet.size();
         int rfCount = 0;
-#ifndef HAVE_STRUCT_STAT_ST_MTIMESPEC
+
         // TODO: replace this with better linux revision ID detection
         int64_t revisionIDplus = 0;
-#endif
+
         for(int f = 0; f < setSize; ++f)
         {
@@ -93 +99 @@
                                 int64_t rid = FileModificationTime(st);
                                 if(rid > revisionID) revisionID = rid;
-#ifndef HAVE_STRUCT_STAT_ST_MTIMESPEC
                                 revisionIDplus += st.st_size;
-#endif
                         }
                 }
@@ -102 +106 @@
         {
                 (*pRevisionID) = revisionID;
-#ifndef HAVE_STRUCT_STAT_ST_MTIMESPEC
+#ifdef NDEBUG
+                // The resolution of timestamps may be very low, e.g.
+                // 1 second. So add the size to it to give a bit more
+                // chance of it changing.
+                // TODO: Make this better.
+                // Disabled in debug mode, to simulate filesystem with
+                // 1-second timestamp resolution, e.g. MacOS X HFS, Linux.
                 (*pRevisionID) += revisionIDplus;
 #endif