[
https://issues.apache.org/jira/browse/NUTCH-522?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12513895
]
Doğacan Güney commented on NUTCH-522:
-------------------------------------
I like the idea, but your patch seems to have a bug. Now injector only injects
a url if it is *not* valid.
Injector.java:75: if (!validator.isValid(url)) {
I think you should put a return there instead of moving normalizing and
filtering code into that branch.
> Use URLValidator in the Injector
> --------------------------------
>
> Key: NUTCH-522
> URL: https://issues.apache.org/jira/browse/NUTCH-522
> Project: Nutch
> Issue Type: Improvement
> Components: injector
> Reporter: Emmanuel Joke
> Priority: Minor
> Fix For: 1.0.0
>
> Attachments: NUTCH-522.patch
>
>
> Same as NUTCH-505, we should use the UrlValidator to check url in the Injector
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
-------------------------------------------------------------------------
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-developers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/nutch-developers
