[ http://issues.apache.org/jira/browse/NUTCH-227?page=comments#action_12369660 ]
Andrzej Bialecki commented on NUTCH-227: ----------------------------------------- Isn't it so that QueryFilter (which is an interface) already extends Configurable? What seems to be missing in QueryFilters.java is a call to setConf(), around line 73. > Basic Query Filter no more uses Configuration > --------------------------------------------- > > Key: NUTCH-227 > URL: http://issues.apache.org/jira/browse/NUTCH-227 > Project: Nutch > Type: Bug > Components: searcher > Versions: 0.8-dev > Reporter: Jerome Charron > Assignee: Jerome Charron > Fix For: 0.8-dev > > Since NUTCH-169, the BasicIndexingFilter has no way to retrieve its > configuration parameters (query.url.boost, query.anchor.boost, > query.title.boost, query.host.boost, query.phrase.boost) : The > setConf(Configuration) method is never called by the QueryFilters class. > More generaly, we should provide a way for QueryFilter to be Configurable. > Two solutions: > 1. The QueryFilters checks that a QueryFilter implements Configurable and > then call the setConf() method. > 2. QueryFilter extends Configurable => all QueryFilter must implement > Configurable. > My preference goes to 1, and if there is no objection, I will commit a patch > in the next few days. -- 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
