Changeset 18615 for indexers/trunk

Timestamp:

2009-03-03T13:59:17+13:00 (15 years ago)

Author:

mdewsnip

Message:

Changed GS2LuceneQuery.java so it now catches exceptions thrown trying to write the query result cache file and warns about them, but doesn't bother with the full stack trace. It won't affect the functionality if we can't write some cache files, it will just affect the speed of subsequent requests. Example exceptions are "permission denied" errors, or "filename too long" errors (the filter string can get very long in some collections). By Michael Dewsnip at DL Consulting Ltd.

File:

• indexers/trunk/lucene-gs/src/org/greenstone/LuceneWrapper/GS2LuceneQuery.java (modified) (1 diff)

indexers/trunk/lucene-gs/src/org/greenstone/LuceneWrapper/GS2LuceneQuery.java

@@ -626 +626 @@
     // Cache this query result, if desired
     if (query_result_caching_enabled) {
-        FileWriter query_result_cache_file_writer = new FileWriter(query_result_cache_file);
-        query_result_cache_file_writer.write(query_results_xml.toString());
-        query_result_cache_file_writer.close();
+        // Catch any exceptions thrown trying to write the query result cache file and warn about them, but don't
+        //   bother with the full stack trace. It won't affect the functionality if we can't write some cache
+        //   files, it will just affect the speed of subsequent requests.
+        // Example exceptions are "permission denied" errors, or "filename too long" errors (the filter string
+        //   can get very long in some collections)
+        try
+        {
+        FileWriter query_result_cache_file_writer = new FileWriter(query_result_cache_file);
+        query_result_cache_file_writer.write(query_results_xml.toString());
+        query_result_cache_file_writer.close();
+        }
+        catch (Exception exception)
+        {
+        System.err.println("Warning: Exception occurred trying to write query result cache file (" + exception + ")");
+        }
     }
     }