Changeset 10652 in josm for trunk/src

Timestamp:

2016-07-26T22:06:52+02:00 (8 years ago)

Author:

wiktorn

Message:

Fix not using the calculated disk cache size.

Add unit tests for #13128 - using existing file size if greater than desired cache size.

Closes #13128

File:

• trunk/src/org/openstreetmap/josm/data/cache/JCSCacheManager.java (modified) (4 diffs)

trunk/src/org/openstreetmap/josm/data/cache/JCSCacheManager.java

@@ -45 +45 @@
     private static long maxObjectTTL = -1;
     private static final String PREFERENCE_PREFIX = "jcs.cache";
-    public static BooleanProperty USE_BLOCK_CACHE = new BooleanProperty(PREFERENCE_PREFIX + ".use_block_cache", true);
+    public static final BooleanProperty USE_BLOCK_CACHE = new BooleanProperty(PREFERENCE_PREFIX + ".use_block_cache", true);
 
     private static final AuxiliaryCacheFactory diskCacheFactory =
@@ -173 +173 @@
         if (cachePath != null && cacheDirLock != null) {
             IDiskCacheAttributes diskAttributes = getDiskCacheAttributes(maxDiskObjects, cachePath, cacheName);
-            /*
-             * BlockDiskCache never optimizes the file, so when file size is reduced, it will never be truncated to desired size.
-             *
-             * If for some mysterious reason, file size is greater than the value set in preferences, just use the whole file. If the user
-             * wants to reduce the file size, (s)he may just go to preferences and there it should be handled (by removing old file)
-             */
-            if (USE_BLOCK_CACHE.get()) {
-                File diskCacheFile = new File(cachePath + File.separator + diskAttributes.getCacheName() + ".data");
-                maxDiskObjects = (int) Math.max(maxDiskObjects, diskCacheFile.length()/1024);
-            }
             try {
                 if (cc.getAuxCaches().length == 0) {
@@ -210 +200 @@
     private static IDiskCacheAttributes getDiskCacheAttributes(int maxDiskObjects, String cachePath, String cacheName) {
         IDiskCacheAttributes ret;
+        removeStaleFiles(cachePath + File.separator + cacheName, (USE_BLOCK_CACHE.get() ? "_INDEX_v2" : "_BLOCK_v2"));
+        cacheName = cacheName + (USE_BLOCK_CACHE.get() ? "_BLOCK_v2" : "_INDEX_v2");
+
         if (USE_BLOCK_CACHE.get()) {
             BlockDiskCacheAttributes blockAttr = new BlockDiskCacheAttributes();
-            blockAttr.setMaxKeySize(maxDiskObjects);
+            /*
+             * BlockDiskCache never optimizes the file, so when file size is reduced, it will never be truncated to desired size.
+             *
+             * If for some mysterious reason, file size is greater than the value set in preferences, just use the whole file. If the user
+             * wants to reduce the file size, (s)he may just go to preferences and there it should be handled (by removing old file)
+             */
+            File diskCacheFile = new File(cachePath + File.separator + cacheName + ".data");
+            if (diskCacheFile.exists()) {
+                blockAttr.setMaxKeySize((int) Math.max(maxDiskObjects, diskCacheFile.length()/1024));
+            } else {
+                blockAttr.setMaxKeySize(maxDiskObjects);
+            }
             blockAttr.setBlockSizeBytes(4096); // use 4k blocks
             ret = blockAttr;
@@ -227 +231 @@
             ret.setDiskPath(cachePath);
         }
-        ret.setCacheName(cacheName + (USE_BLOCK_CACHE.get() ? "_BLOCK_v2" : "_INDEX_v2"));
-
-        removeStaleFiles(cachePath + File.separator + cacheName, (USE_BLOCK_CACHE.get() ? "_INDEX_v2" : "_BLOCK_v2"));
+        ret.setCacheName(cacheName);
+
         return ret;
     }