lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris Male (JIRA)" <>
Subject [jira] [Updated] (LUCENE-4048) Move getLines out of ResourceLoader and require Charset
Date Mon, 14 May 2012 04:55:51 GMT


Chris Male updated LUCENE-4048:

    Attachment: LUCENE-4048.patch

Patch with a bunch of nocommits for things I'm not happy about.  

I've added the line reading functionality from {{SolrResourceLoader}} to WordlistLoader but
I'm unsure that it belongs there since it's about reading the contents of generic resources,
not just those related to stopwords.

What I'm thinking of doing is changing ResourceLoader to be an abstract class and putting
the contents reading functionality there, and I can also put the boilerplate of integrating
it with resources there too.
> Move getLines out of ResourceLoader and require Charset
> -------------------------------------------------------
>                 Key: LUCENE-4048
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: modules/analysis
>            Reporter: Chris Male
>         Attachments: LUCENE-4048.patch
> {{ResourceLoader.getLines()}} is only used by analysis factories.  {{SolrResourceLoader}}'s
implementation does the job well and it's unlikely that another {{ResourceLoader}} implementation
would handle it differently.
> We should extract the {{getLines()}} method out to {{AbstractAnalysisFactory}} so it
can be used by the factories.  Additionally we shouldn't assume the files are encoded in UTF-8,
instead we should allow a Charset to be specified.
> This would take us one step closer to reducing the {{ResourceLoader}} interface just
to what it says, a loader of resources.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:!default.jspa
For more information on JIRA, see:


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message