Ticket #15825: patch_for_15825.diff

django/core/cache/backends/filebased.py

@@ -4 +4 @@
 import os
 import shutil
 import time
+import random
 try:
     import cPickle as pickle
 except ImportError:
@@ -113 +114 @@
             return
 
         try:
-            filelist = sorted(os.listdir(self._dir))
+            filelist = os.listdir(self._dir)
         except (IOError, OSError):
             return
 
         if self._cull_frequency == 0:
-            doomed = filelist
-        else:
-            doomed = [os.path.join(self._dir, k) for (i, k) in enumerate(filelist) if i % self._cull_frequency == 0]
+            self.clear()
+            return
+
+        ranno = random.randint(0, self._cull_frequency)
+        doomed = [
+            os.path.join(self._dir, k)
+            for (i, k) in enumerate(filelist)
+            if i % self._cull_frequency == ranno]
 
         for topdir in doomed:
             try:
-                for root, _, files in os.walk(topdir):
-                    for f in files:
-                        self._delete(os.path.join(root, f))
+                shutil.rmtree(topdir)
             except (IOError, OSError):
                 pass
 

tests/regressiontests/cache/tests.py

@@ -414 +414 @@
         number of entries will vary between backends"""
         # Create initial cache key entries. This will overflow the cache, causing a cull
         for i in range(1, initial_count):
-            self.cache.set('cull%d' % i, 'value', 1000)
+            self.cache.set('cull-%d' % i, 'value', 1000)
         count = 0
         # Count how many keys are left in the cache.
         for i in range(1, initial_count):
-            if self.cache.has_key('cull%d' % i):
+            if self.cache.has_key('cull-%d' % i):
                 count = count + 1
         self.assertEqual(count, final_count)
 
@@ -880 +880 @@
         self.assertTrue(not os.path.exists(os.path.dirname(os.path.dirname(keypath))))
 
     def test_cull(self):
-        self.perform_cull_test(50, 29)
+        self.perform_cull_test(35, 24)
 
     def test_old_initialization(self):
         self.cache = get_cache('file://%s?max_entries=30' % self.dirname)
-        self.perform_cull_test(50, 29)
+        self.perform_cull_test(35, 24)
 
 class CustomCacheKeyValidationTests(unittest.TestCase):
     """