Changeset 13643 in josm for trunk/src/org/openstreetmap/josm/data/cache/JCSCacheManager.java

Timestamp:

2018-04-16T23:00:30+02:00 (6 years ago)

Author:

wiktorn

Message:

In case of any IOExceptions when creating disk cache return memory-only cache

Closes: #16193

File:

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

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

@@ -101 +101 @@
 
     @SuppressWarnings("resource")
-    private static void initialize() throws IOException {
+    private static void initialize()  {
         File cacheDir = new File(Config.getDirs().getCacheDirectory(true), "jcs");
 
-        if (!cacheDir.exists() && !cacheDir.mkdirs())
-            throw new IOException("Cannot access cache directory");
-
-        File cacheDirLockPath = new File(cacheDir, ".lock");
-        if (!cacheDirLockPath.exists() && !cacheDirLockPath.createNewFile()) {
-            Logging.warn("Cannot create cache dir lock file");
-        }
-        cacheDirLock = FileChannel.open(cacheDirLockPath.toPath(), StandardOpenOption.WRITE).tryLock();
-
-        if (cacheDirLock == null)
-            Logging.warn("Cannot lock cache directory. Will not use disk cache");
-
+        if (!cacheDir.exists() && !cacheDir.mkdirs()) {
+            Logging.warn("Cache directory " + cacheDir.toString() + " does not exists and could not create it");
+        } else {
+            File cacheDirLockPath = new File(cacheDir, ".lock");
+            try {
+                if (!cacheDirLockPath.exists() && !cacheDirLockPath.createNewFile()) {
+                    Logging.warn("Cannot create cache dir lock file");
+                }
+                cacheDirLock = FileChannel.open(cacheDirLockPath.toPath(), StandardOpenOption.WRITE).tryLock();
+
+                if (cacheDirLock == null)
+                    Logging.warn("Cannot lock cache directory. Will not use disk cache");
+            } catch (IOException e) {
+                Logging.warn("Cannot create cache dir \"" + cacheDirLockPath.toString() + "\" lock file: " + e.toString());
+                Logging.warn("Will not use disk cache");
+            }
+        }
         // this could be moved to external file
         Properties props = new Properties();
@@ -142 +147 @@
      * @param cacheName region name
      * @return cache access object
-     * @throws IOException if directory is not found
-     */
-    public static <K, V> CacheAccess<K, V> getCache(String cacheName) throws IOException {
+     */
+    public static <K, V> CacheAccess<K, V> getCache(String cacheName) {
         return getCache(cacheName, DEFAULT_MAX_OBJECTS_IN_MEMORY.get().intValue(), 0, null);
     }
@@ -157 +161 @@
      * @param cachePath         path to disk cache. if null, no disk cache will be created
      * @return cache access object
-     * @throws IOException if directory is not found
-     */
-    public static <K, V> CacheAccess<K, V> getCache(String cacheName, int maxMemoryObjects, int maxDiskObjects, String cachePath)
-            throws IOException {
+     */
+    public static <K, V> CacheAccess<K, V> getCache(String cacheName, int maxMemoryObjects, int maxDiskObjects, String cachePath) {
         if (cacheManager != null)
             return getCacheInner(cacheName, maxMemoryObjects, maxDiskObjects, cachePath);
@@ -172 +174 @@
 
     @SuppressWarnings("unchecked")
-    private static <K, V> CacheAccess<K, V> getCacheInner(String cacheName, int maxMemoryObjects, int maxDiskObjects, String cachePath)
-            throws IOException {
+    private static <K, V> CacheAccess<K, V> getCacheInner(String cacheName, int maxMemoryObjects, int maxDiskObjects, String cachePath) {
         CompositeCache<K, V> cc = cacheManager.getCache(cacheName, getCacheAttributes(maxMemoryObjects));
 
@@ -183 +184 @@
                             diskAttributes, cacheManager, null, new StandardSerializer())});
                 }
-            } catch (IOException e) {
-                throw e;
-            } catch (Exception e) { // NOPMD
-                throw new IOException(e);
+            } catch (Exception e) {
+                // in case any error in setting auxiliary cache, do not use disk cache at all - only memory
+                cc.setAuxCaches(new AuxiliaryCache[0]);
             }
         }