You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nutch.apache.org by Nicolás Lichtmaier <ni...@reloco.com.ar> on 2007/06/01 22:38:22 UTC
[PATCH] Moving HitDetails construction to a HitDetails constructor
(v2).
This is a fixed version of the previous patch. Please, don't ignore me
=). I'm trying to use Lucene queries with Nutch and this patch will
help. This patch also removes a deprecated API usage, removes useless
object creation and array copying.
Thanks!
Re: [PATCH] Moving HitDetails construction to a HitDetails constructor
(v2).
Posted by Nicolás Lichtmaier <ni...@reloco.com.ar>.
>> Please, don't ignore me =).
> We don't - but there's only so much ou can do in 24 hrs/day, and Nutch
> developers have their own lives to attend to ... ;)
=) Sorry, I didn't mean to sound "demanding". It's that there's a
natural focus in real features and I thought that "tidyness" patches get
unnoticed.
>
>> I'm trying to use Lucene queries with Nutch and this patch will help.
>> This patch also removes a deprecated API usage, removes useless
>> object creation and array copying.
>
> I believe the conversion from Document to HitDetails was separated
> this way on purpose. Please note that front-end Nutch API has no
> dependencies on Lucene classes. If we applied your patch, all of a
> sudden HitDetails would become dependent on Lucene, causing front-end
> applications to become dependent on Lucene, too.
>
> We can certainly fix the use of deprecated API as you suggested. As
> for the rest of the patch, in my opinion it should not be applied.
>
Oh, I see... a pitty. It looked cleaner too me, and I'll have to
copy+paste that into my code. What about the other patch? (Retrofit Hits
to implement List)
Re: [PATCH] Moving HitDetails construction to a HitDetails constructor
(v2).
Posted by Andrzej Bialecki <ab...@getopt.org>.
Nicolás Lichtmaier wrote:
> This is a fixed version of the previous patch.
In the future, please use JIRA bug tracking system to submit patches.
> Please, don't ignore me =).
We don't - but there's only so much ou can do in 24 hrs/day, and Nutch
developers have their own lives to attend to ... ;)
> I'm trying to use Lucene queries with Nutch and this patch will
> help. This patch also removes a deprecated API usage, removes useless
> object creation and array copying.
I believe the conversion from Document to HitDetails was separated this
way on purpose. Please note that front-end Nutch API has no dependencies
on Lucene classes. If we applied your patch, all of a sudden HitDetails
would become dependent on Lucene, causing front-end applications to
become dependent on Lucene, too.
We can certainly fix the use of deprecated API as you suggested. As for
the rest of the patch, in my opinion it should not be applied.
--
Best regards,
Andrzej Bialecki <><
___. ___ ___ ___ _ _ __________________________________
[__ || __|__/|__||\/| Information Retrieval, Semantic Web
___|||__|| \| || | Embedded Unix, System Integration
http://www.sigram.com Contact: info at sigram dot com