[jira] [Commented] (NUTCH-2602) Configuration values in the description of index writers

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (NUTCH-2602) Configuration values in the description of index writers

JIRA jira@apache.org

    [ https://issues.apache.org/jira/browse/NUTCH-2602?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16536894#comment-16536894 ]

Sebastian Nagel commented on NUTCH-2602:
----------------------------------------

+1 to show the concrete configuration for index writer plugins instead of listing available properties. I'm a little bit concerned about removing the descriptions from the code at all (they're now only in the [wiki|https://wiki.apache.org/nutch/IndexWriters]). That's good for now, but as the wiki tends to get outdated over time and, in general, is not bound to a specific Nutch version, it would be good to document at least those options which are not self-explanatory, either by adding Javadoc comments to the constants or comments in indexwriters.xml.template.

> Configuration values in the description of index writers
> --------------------------------------------------------
>
>                 Key: NUTCH-2602
>                 URL: https://issues.apache.org/jira/browse/NUTCH-2602
>             Project: Nutch
>          Issue Type: Improvement
>          Components: indexer, plugin
>    Affects Versions: 1.15
>            Reporter: Roannel Fernández Hernández
>            Assignee: Roannel Fernández Hernández
>            Priority: Minor
>             Fix For: 1.16
>
>
> Since [GitHub Pull Request #218|https://github.com/apache/nutch/pull/218] when you have 2+ different configuration of the same index writers (the same implementation class), the index command print the same description several times. I propose the {{describe()}} method show the values of its own configuration and not a generic one.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)