[
https://issues.apache.org/jira/browse/SOLR-5264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13781190#comment-13781190
]
Steve Rowe commented on SOLR-5264:
----------------------------------
The idea seems good to me. All Solr tests pass for me on trunk with the patch
applied.
A few nits:
* removeAll and removeArgs don't look very different, but they do different
things. I think their javadocs should mention each other and say how they're
different from each other.
* I don't think removeArgs is a sufficiently descriptive name, but all attempts
I've made to come up with a better one are way too long, like
recursivelyRemoveAllStringArgs. But I think we could do better somehow here.
oneOrMany is short but doesn't mention that the args are removed, so I don't
think we should just use it.
* The deprecation message on oneOrMany should say which version it will be
removed in. I think 5.0 is a better choice than 4.7 or 4.8, since there is
already established precedent for deprecation in major version X, removal in
major version X+1. Then the trunk commit will remove it, and the branch_4x
version will have the deprecation message, and there is nothing else to do
(except a CHANGES.txt entry explaining this, which you should include when you
commit), so this strategy is also easier to implement than removal in X.Y+1 or
2, since you have to remember to do that then.
> New method on NamedList to return one or many config arguments as collection
> ----------------------------------------------------------------------------
>
> Key: SOLR-5264
> URL: https://issues.apache.org/jira/browse/SOLR-5264
> Project: Solr
> Issue Type: Improvement
> Components: clients - java
> Affects Versions: 4.5
> Reporter: Shawn Heisey
> Assignee: Shawn Heisey
> Priority: Minor
> Fix For: 5.0, 4.6
>
> Attachments: SOLR-5264.patch, SOLR-5264.patch, SOLR-5264.patch,
> SOLR-5264.patch
>
>
> In the FieldMutatingUpdateProcessorFactory is a method called "oneOrMany"
> that takes all of the entries in a NamedList and pulls them out into a
> Collection. I'd like to use that in a custom update processor I'm building.
> It seems as though this functionality would be right at home as part of
> NamedList itself. Here's a patch that moves the method.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
