Gian, Even if you can't write a failing unit test (if you did it would be awesome), please open a JIRA issue on this and attach your patch. Also, you may want to try 4.0 as opposed to 3.6 as some of the 3.6 issues with DIH are resolved in 4.0.
https://issues.apache.org/jira/secure/Dashboard.jspa James Dyer E-Commerce Systems Ingram Content Group (615) 213-4311 -----Original Message----- From: Mikhail Khludnev [mailto:mkhlud...@griddynamics.com] Sent: Friday, September 21, 2012 12:03 PM To: solr-user@lucene.apache.org Subject: Re: DIH problem Gian, The only way to handle it is to provide a test case and attach to jira. Thanks On Fri, Sep 21, 2012 at 6:03 PM, Gian Marco Tagliani <gm.tagli...@gmail.com>wrote: > Hi, > I'm updating my Solr from version 3.4 to version 3.6.1 and I'm facing a > little problem with the DIH. > > In the delta-import I'm using the /parentDeltaQuery/ feature of the DIH to > update the parent entity. > I don't think this is working properly. > > I realized that it's just executing the /parentDeltaQuery/ with the first > record of the /deltaQuery /result. > Comparing the code with the previous versions I noticed that the > rowIterator was never set to null. > > To solve this I wrote a simple patch: > > ----------------------------- > Index: solr/contrib/**dataimporthandler/src/java/** > org/apache/solr/handler/**dataimport/**EntityProcessorBase.java > ==============================**==============================**======= > --- solr/contrib/**dataimporthandler/src/java/**org/apache/solr/handler/** > dataimport/**EntityProcessorBase.java (revision 31454) > +++ solr/contrib/**dataimporthandler/src/java/**org/apache/solr/handler/** > dataimport/**EntityProcessorBase.java (working copy) > @@ -121,6 +121,7 @@ > if (rowIterator.hasNext()) > return rowIterator.next(); > query = null; > + rowIterator = null; > return null; > } catch (Exception e) { > SolrException.log(log, "getNext() failed for query '" + query + > "'", e); > ----------------------------- > > > Do you think this is correct? > > Thanks for your help > > -- > Gian Marco Tagliani > > > > -- Sincerely yours Mikhail Khludnev Tech Lead Grid Dynamics <http://www.griddynamics.com> <mkhlud...@griddynamics.com>
