You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nutch.apache.org by "Sami Siren (JIRA)" <ji...@apache.org> on 2007/05/09 18:39:15 UTC
[jira] Updated: (NUTCH-469) changes to geoPosition plugin to make
it work on nutch 0.9
[ https://issues.apache.org/jira/browse/NUTCH-469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sami Siren updated NUTCH-469:
-----------------------------
Attachment: NUTCH-469-2007-05-09.txt.gz
tnahks for putting this together, I briefly checked through the .gz and patch
-please use diffs against trunk in future, they're more easy to check (svn diff > file)
-there is no junit tests at all, however there is tiny piece of test code in class GeoIndexingFilter, atleast this code could perhaps be moved to a junit test class
-i replaced System.out.prints with logging statements
-i changed some formatting
-would it make sense to move the zip folder from conf to under plugins src/java and change the load mechanism to use (context) class loader as i believe they are quite static piece of information once generated?
I am attaching the patch is it is now
> changes to geoPosition plugin to make it work on nutch 0.9
> ----------------------------------------------------------
>
> Key: NUTCH-469
> URL: https://issues.apache.org/jira/browse/NUTCH-469
> Project: Nutch
> Issue Type: Improvement
> Components: indexer, searcher
> Affects Versions: 0.9.0
> Reporter: Mike Schwartz
> Fix For: 0.7.3
>
> Attachments: geoPosition-0.5.tgz, geoPosition0.6_cdiff.zip, NUTCH-469-2007-05-09.txt.gz
>
>
> I have modified the geoPosition plugin (http://wiki.apache.org/nutch/GeoPosition) code to work with nutch 0.9. (The code was built originally using nutch 0.7.) I'd like to contribute my changes back to the nutch project. I already communicated with the code's author (Matthias Jaekle), and he agrees with my mods.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
Re: [jira] Updated: (NUTCH-469) changes to geoPosition plugin
to make it work on nutch 0.9
Posted by Mike Schwartz <mf...@gmail.com>.
Sami,
Thanks for your response. In answer to the points you made:
>please use diffs against trunk in future, they're more easy to check
>(svn diff > file)
will do
>there is no junit tests at all, however there is tiny piece of test code
>in class GeoIndexingFilter, atleast this code could perhaps be moved to
>a junit test class
>
>i replaced System.out.prints with logging statements
both of these issues predate my changes - I didn't write the original
code, I just updated it to get it to work on nutch 0.9.
>i changed some formatting
>
>would it make sense to move the zip folder from conf to under plugins
>src/java and change the load mechanism to use (context) class loader as
>i believe they are quite static piece of information once generated?
I agree, but again it wasn't my code so if you're looking for the
original author's inputs we should ask him. I'll forward this
exchange to him (I don't think he subscribes to nutch-dev)
thanks
- Mike
At 10:39 AM 5/9/2007, you wrote:
> [
> https://issues.apache.org/jira/browse/NUTCH-469?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
> ]
>
>Sami Siren updated NUTCH-469:
>-----------------------------
>
> Attachment: NUTCH-469-2007-05-09.txt.gz
>
>tnahks for putting this together, I briefly checked through the .gz and patch
>
>-please use diffs against trunk in future, they're more easy to
>check (svn diff > file)
>-there is no junit tests at all, however there is tiny piece of test
>code in class GeoIndexingFilter, atleast this code could perhaps be
>moved to a junit test class
>-i replaced System.out.prints with logging statements
>-i changed some formatting
>-would it make sense to move the zip folder from conf to under
>plugins src/java and change the load mechanism to use (context)
>class loader as i believe they are quite static piece of information
>once generated?
>
>I am attaching the patch is it is now
>
> > changes to geoPosition plugin to make it work on nutch 0.9
> > ----------------------------------------------------------
> >
> > Key: NUTCH-469
> > URL: https://issues.apache.org/jira/browse/NUTCH-469
> > Project: Nutch
> > Issue Type: Improvement
> > Components: indexer, searcher
> > Affects Versions: 0.9.0
> > Reporter: Mike Schwartz
> > Fix For: 0.7.3
> >
> > Attachments: geoPosition-0.5.tgz,
> geoPosition0.6_cdiff.zip, NUTCH-469-2007-05-09.txt.gz
> >
> >
> > I have modified the geoPosition plugin
> (http://wiki.apache.org/nutch/GeoPosition) code to work with nutch
> 0.9. (The code was built originally using nutch 0.7.) I'd like to
> contribute my changes back to the nutch project. I already
> communicated with the code's author (Matthias Jaekle), and he
> agrees with my mods.
>
>--
>This message is automatically generated by JIRA.
>-
>You can reply to this email to add a comment to the issue online.