[jira] [Updated] (NUTCH-1990) Use URI.normalise() in BasicURLNormalizer

["Sebastian Nagel (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/NUTCH-1990?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sebastian Nagel updated NUTCH-1990:
-----------------------------------
    Attachment: NUTCH-1990-trial1.patch

Sounds reasonable and would "en passant" resolve NUTCH-1062. But a trial (attached) was only partially successful because URI.normalize() does not
- remove leading "/.." (should be definitely done according [RFC 3986, section 5.2.4, 2A|http://tools.ietf.org/html/rfc3986#section-5.2.4])
- add a "/" if path is/gets empty

After adding these two steps unit tests passed, except one which in fact was wrong before ("http://foo.com/aa/.." should be normalized to "http://foo.com/").
The patch is definitely not ready: the old version checks first whether there is a path element to be normalized which can have massive performance impact.
In any case, we should compare old and new version regarding quality and performance with a larger set of unnormalized outlinks.

> Use URI.normalise() in BasicURLNormalizer
> -----------------------------------------
>
>                 Key: NUTCH-1990
>                 URL: https://issues.apache.org/jira/browse/NUTCH-1990
>             Project: Nutch
>          Issue Type: Improvement
>    Affects Versions: 1.9
>            Reporter: Julien Nioche
>            Assignee: Julien Nioche
>         Attachments: NUTCH-1990-trial1.patch
>
>
> One of the things that [BasicURLNormalizer|https://github.com/apache/nutch/blob/trunk/src/plugin/urlnormalizer-basic/src/java/org/apache/nutch/net/urlnormalizer/basic/BasicURLNormalizer.java] is to remove unnecessary dot segments in path.
> Instead of implementing the logic ourselves with some antiquated regex library, we should simply use [http://docs.oracle.com/javase/7/docs/api/java/net/URI.html#normalize()] which does the same and is probably more efficient.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)