Ok, I'll try to verify if there is the same issue in the 4.0 and I'll open the issue in Jira.
thanks -- Gian Marco On Sat, Sep 22, 2012 at 9:34 PM, Dyer, James <james.d...@ingramcontent.com>wrote: > 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> > >
