Author: shalin
Date: Thu Dec 10 07:54:12 2009
New Revision: 889127
URL: http://svn.apache.org/viewvc?rev=889127&view=rev
Log:
SOLR-1639 -- Fixed misleading error message when dataimport.properties is not
writable
Modified:
lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt
lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/DocBuilder.java
Modified: lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt
URL:
http://svn.apache.org/viewvc/lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt?rev=889127&r1=889126&r2=889127&view=diff
==============================================================================
--- lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt (original)
+++ lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt Thu Dec 10 07:54:12
2009
@@ -34,6 +34,7 @@
* SOLR-1638: Fixed NullPointerException during import if uniqueKey is not
specified
in schema (Akshay Ukey via shalin)
+* SOLR-1639: Fixed misleading error message when dataimport.properties is not
writable (shalin)
Other Changes
----------------------
Modified:
lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/DocBuilder.java
URL:
http://svn.apache.org/viewvc/lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/DocBuilder.java?rev=889127&r1=889126&r2=889127&view=diff
==============================================================================
---
lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/DocBuilder.java
(original)
+++
lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/DocBuilder.java
Thu Dec 10 07:54:12 2009
@@ -229,7 +229,13 @@
if (requestParameters.optimize)
addStatusMessage("Optimized");
}
- writer.persist(lastIndexTimeProps);
+ try {
+ writer.persist(lastIndexTimeProps);
+ } catch (Exception e) {
+ LOG.error("Could not write property file", e);
+ statusMessages.put("error", "Could not write property file. Delta
imports will not work. " +
+ "Make sure your conf directory is writable");
+ }
}
void rollback() {