[
https://issues.apache.org/jira/browse/SOLR-11913?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16415996#comment-16415996
]
David Smiley commented on SOLR-11913:
-------------------------------------
{quote}I tried doing so but it gave me some error stating _Rat problems were
found!_ . I tried searching over the internet but couldn't find anything
useful, did some playing around build.xml but all was in waste.
{quote}
That refers to quality or style checks that Lucene/Solr expressly insists upon.
The log output _should_ tell you exactly what the problem is.
{quote} when you said about callers of getParameterNamesiterator(), did you
mean callers in SolrParams class only or all callers of that iterator function?
{quote}
All. Note your IDE should have a "find usages" or similarly named feature.
bq. Though ModifiableSolrParams is just an example, I check for this class and
it is using getParameterNamesiterator() for its add(SolrParams params) function
which has never been called, so no point in changing that, I guess.
Please do change it. When you say "has never been called"; maybe I don't
understand what you mean but IntelliJ reports 30 usages across the codebase.
> SolrParams ought to implement Iterable<Map.Entry<String,String[]>>
> ------------------------------------------------------------------
>
> Key: SOLR-11913
> URL: https://issues.apache.org/jira/browse/SOLR-11913
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Reporter: David Smiley
> Priority: Minor
> Labels: newdev
> Attachments: SOLR-11913.patch, SOLR-11913.patch, SOLR-11913_v2.patch
>
>
> SolrJ ought to implement {{Iterable<Map.Entry<String,String[]>>}} so that
> it's easier to iterate on it, either using Java 5 for-each style, or Java 8
> streams. The implementation on ModifiableSolrParams can delegate through to
> the underlying LinkedHashMap entry set. The default impl can produce a
> Map.Entry with a getValue that calls through to getParams.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
