You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by sh...@apache.org on 2009/03/09 15:55:44 UTC

svn commit: r751696 - in /lucene/solr/trunk/contrib/dataimporthandler: CHANGES.txt src/main/java/org/apache/solr/handler/dataimport/SolrWriter.java

Author: shalin
Date: Mon Mar  9 14:55:40 2009
New Revision: 751696

URL: http://svn.apache.org/viewvc?rev=751696&view=rev
Log:
SOLR-1053 -- IndexOutOfBoundsException in SolrWriter.getResourceAsString when size of data-config.xml is a multiple of 1024 bytes

Modified:
    lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt
    lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/SolrWriter.java

Modified: lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt?rev=751696&r1=751695&r2=751696&view=diff
==============================================================================
--- lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt (original)
+++ lucene/solr/trunk/contrib/dataimporthandler/CHANGES.txt Mon Mar  9 14:55:40 2009
@@ -161,6 +161,10 @@
 21.SOLR-1042: Fix memory leak in DIH by making TemplateString non-static member in VariableResolverImpl
               (Ryuuichi Kumai via shalin)
 
+22.SOLR-1053: IndexOutOfBoundsException in SolrWriter.getResourceAsString when size of data-config.xml is a
+              multiple of 1024 bytes.
+              (Herb Jiang via shalin)
+
 Documentation
 ----------------------
 

Modified: lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/SolrWriter.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/SolrWriter.java?rev=751696&r1=751695&r2=751696&view=diff
==============================================================================
--- lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/SolrWriter.java (original)
+++ lucene/solr/trunk/contrib/dataimporthandler/src/main/java/org/apache/solr/handler/dataimport/SolrWriter.java Mon Mar  9 14:55:40 2009
@@ -192,11 +192,8 @@
     byte[] buf = new byte[1024];
     int sz = 0;
     try {
-      while (true) {
-        sz = in.read(buf);
+      while ((sz = in.read(buf)) != -1) {
         baos.write(buf, 0, sz);
-        if (sz < buf.length)
-          break;
       }
     } finally {
       try {