Browse Source

fix #486 : ignore error

Shinsuke Sugaya 9 years ago
parent
commit
03cbf858ef

+ 3 - 4
src/main/java/org/codelibs/fess/ds/impl/CsvDataStoreImpl.java

@@ -266,12 +266,11 @@ public class CsvDataStoreImpl extends AbstractDataStoreImpl {
                     }
                     final FailureUrlService failureUrlService = ComponentUtil.getComponent(FailureUrlService.class);
                     failureUrlService.store(dataConfig, errorName, url, target);
-                } catch (final Exception | OutOfMemoryError e) {
+                } catch (final Throwable t) {
+                    logger.warn("Crawling Access Exception at : " + dataMap, t);
                     final String url = csvFile.getAbsolutePath() + ":" + csvReader.getLineNumber();
                     final FailureUrlService failureUrlService = ComponentUtil.getComponent(FailureUrlService.class);
-                    failureUrlService.store(dataConfig, e.getClass().getCanonicalName(), url, e);
-
-                    logger.warn("Crawling Access Exception at : " + dataMap, e);
+                    failureUrlService.store(dataConfig, t.getClass().getCanonicalName(), url, t);
                 }
 
                 if (readInterval > 0) {

+ 3 - 4
src/main/java/org/codelibs/fess/ds/impl/EsDataStoreImpl.java

@@ -210,12 +210,11 @@ public class EsDataStoreImpl extends AbstractDataStoreImpl {
                         }
                         final FailureUrlService failureUrlService = ComponentUtil.getComponent(FailureUrlService.class);
                         failureUrlService.store(dataConfig, errorName, url, target);
-                    } catch (final Exception e) {
+                    } catch (final Throwable t) {
+                        logger.warn("Crawling Access Exception at : " + dataMap, t);
                         final String url = hit.getIndex() + "/" + hit.getType() + "/" + hit.getId();
                         final FailureUrlService failureUrlService = ComponentUtil.getComponent(FailureUrlService.class);
-                        failureUrlService.store(dataConfig, e.getClass().getCanonicalName(), url, e);
-
-                        logger.warn("Crawling Access Exception at : " + dataMap, e);
+                        failureUrlService.store(dataConfig, t.getClass().getCanonicalName(), url, t);
                     }
 
                     if (bulkRequest != null) {