[Checkins] SVN: zc.FileStorage/trunk/src/zc/FileStorage/__init__.py Make more judicious use of posix_fadvise.

Jim Fulton jim at zope.com
Sun Mar 7 09:27:36 EST 2010


Log message for revision 109783:
  Make more judicious use of posix_fadvise.
  

Changed:
  U   zc.FileStorage/trunk/src/zc/FileStorage/__init__.py

-=-
Modified: zc.FileStorage/trunk/src/zc/FileStorage/__init__.py
===================================================================
--- zc.FileStorage/trunk/src/zc/FileStorage/__init__.py	2010-03-07 13:58:00 UTC (rev 109782)
+++ zc.FileStorage/trunk/src/zc/FileStorage/__init__.py	2010-03-07 14:27:36 UTC (rev 109783)
@@ -299,7 +299,7 @@
 
         logging.info('copy to pack time')
         output = open(self._name + ".pack", "w+b")
-        self._freecache = _freefunc(output)
+        self._freeoutputcache = _freefunc(output)
         index, new_pos = self.copyToPacktime(packpos, index, output)
         if new_pos == packpos:
             # pack didn't free any data.  there's no point in continuing.
@@ -310,6 +310,7 @@
             return
 
         logging.info('copy from pack time')
+        self._freecache = self._freeoutputcache = lambda pos: None
         self.copyFromPacktime(packpos, self.file_end, output, index)
 
         # Save the index so the parent process can use it as a starting point.
@@ -450,7 +451,7 @@
                     output.write(tlen)
                     output.seek(new_pos)
 
-                self._freecache(new_pos)
+                self._freeoutputcache(new_pos)
 
 
             pos += 8
@@ -472,7 +473,7 @@
     def copyFromPacktime(self, input_pos, file_end, output, index):
         while input_pos < file_end:
             input_pos = self._copyNewTrans(input_pos, output, index)
-            self._freecache(output.tell())
+            self._freeoutputcache(output.tell())
         return input_pos
 
 



More information about the checkins mailing list