Doğacan Güney wrote:
>> Good catch! The patch looks good, too - please go ahead. One question:
>> why did you remove the call to finishFetchItem() around line 505?
> 
> Because it seems we already call finishFetchItem in that code path
> just before the switch statement. I have opened NUTCH-495 for this, if
> I am mistaken, just give me a nudge and I will send an updated patch.


Indeed, you're right - I should've checked with the base version, not 
just the patch.



-- 
Best regards,
Andrzej Bialecki     <><
  ___. ___ ___ ___ _ _   __________________________________
[__ || __|__/|__||\/|  Information Retrieval, Semantic Web
___|||__||  \|  ||  |  Embedded Unix, System Integration
http://www.sigram.com  Contact: info at sigram dot com


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Nutch-general mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/nutch-general

Reply via email to