nutch-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (NUTCH-2413) When fetching and parsing together, parameter "parse.filter.urls" is ignored
Date Fri, 25 Aug 2017 16:47:00 GMT


ASF GitHub Bot commented on NUTCH-2413:

sebastian-nagel commented on a change in pull request #216: fix for NUTCH-2413 contributed
by maborec

 File path: src/java/org/apache/nutch/fetcher/
 @@ -695,14 +696,23 @@ private ParseStatus output(Text key, CrawlDatum datum, Content content,
           int validCount = 0;
           // Process all outlinks, normalize, filter and deduplicate
+          // NUTCH-2413 Apply filters or normalizers only if configured
+          URLFilters urlFiltersForOutlinks = null;
+          if (conf.getBoolean("parse.filter.urls", true))
 Review comment:
   Maybe better read the configuration in the constructor of FetcherThread and save it in
a instance variable?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

> When fetching and parsing together, parameter "parse.filter.urls" is ignored
> ----------------------------------------------------------------------------
>                 Key: NUTCH-2413
>                 URL:
>             Project: Nutch
>          Issue Type: Bug
>          Components: fetcher, parser
>    Affects Versions: 1.13
>         Environment: Apache Nutch release 1.13.
>            Reporter: Marcos Bori
>             Fix For: 1.14
> In a situation when we want to:
> (1) Execute the fetch and parse together ("fetcher.parse" setting to "true")
> (2) Avoid applying the URL filters when executing this phase.
> Condition (2) can be configured when parsing is executed as a separate process by setting
"parse.filter.urls" to "false".
> However, this setting ("parse.filter.urls") is ignored when we execute the fetch and
parse phases together. 

This message was sent by Atlassian JIRA

View raw message