You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nutch.apache.org by "Jerome Charron (JIRA)" <ji...@apache.org> on 2006/03/16 14:23:42 UTC
[jira] Commented: (NUTCH-233) wrong regular expression hang reduce
process for ever
[ http://issues.apache.org/jira/browse/NUTCH-233?page=comments#action_12370685 ]
Jerome Charron commented on NUTCH-233:
--------------------------------------
Stefan,
I have created a small unit test for urlfilter-regexp and I doesn't notice any incompatibility in java.util.regex with this regexp. Could you please provide the urls that cause problem so that I can add them to me unit tests.
Thanks
Jérôme
> wrong regular expression hang reduce process for ever
> -----------------------------------------------------
>
> Key: NUTCH-233
> URL: http://issues.apache.org/jira/browse/NUTCH-233
> Project: Nutch
> Type: Bug
> Versions: 0.8-dev
> Reporter: Stefan Groschupf
> Priority: Blocker
> Fix For: 0.8-dev
>
> Looks like that the expression ".*(/.+?)/.*?\1/.*?\1/" in regex-urlfilter.txt wasn't compatible with java.util.regex that is actually used in the regex url filter.
> May be it was missed to change it when the regular expression packages was changed.
> The problem was that until reducing a fetch map output the reducer hangs forever since the outputformat was applying the urlfilter a url that causes the hang.
> 060315 230823 task_r_3n4zga at java.lang.Character.codePointAt(Character.java:2335)
> 060315 230823 task_r_3n4zga at java.util.regex.Pattern$Dot.match(Pattern.java:4092)
> 060315 230823 task_r_3n4zga at java.util.regex.Pattern$Curly.match1(Pattern.java:
> I changed the regular expression to ".*(/[^/]+)/[^/]+\1/[^/]+\1/" and now the fetch job works. (thanks to Grant and Chris B. helping to find the new regex)
> However may people can review it and can suggest improvements, since the old regex would match :
> "abcd/foo/bar/foo/bar/foo/" and so will the new one match it also. But the old regex would also match :
> "abcd/foo/bar/xyz/foo/bar/foo/" which the new regex will not match.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira