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

Gus Heck commented on SOLR-12959:
---------------------------------

[~noble.paul] I agree that it would be very nice to be able to use a standard 
collection class instead of NamedList from a programmer convenience standpoint. 
I've had this thought many times too, but expected such a road to be bumpy and 
controversial and so I didn't start down it :).

It seems from the above so far that 2 things stand in the way (identified thus 
far)
 # [~ab] says the current classes were meant to save memory (something worth 
testing since java has evolved a great deal since they were created, named 
lists are mentioned in SOLR-17 so they clearly predate our bug tracking system)
 # The handling of duplicate keys is a feature. Personally, if I were the user 
I'd want the above example hos gave with accidentally duplicated names to throw 
an error, not give me back something that *I would* probably subsequently throw 
away when I stuffed the results into a map or tried to build a JavaScript 
object out of it...

So it would be necessary to verify performance and find widespread agreement 
that that back compatibility break is feasible and worthwhile 

Also note that the code you quote is using SimpleOrderedMap to override and 
ignore the json.nl setting.

> Deprecate solrj SimpleOrderedMap
> --------------------------------
>
>                 Key: SOLR-12959
>                 URL: https://issues.apache.org/jira/browse/SOLR-12959
>             Project: Solr
>          Issue Type: Improvement
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: SolrJ
>            Reporter: Noble Paul
>            Priority: Minor
>
> There is no difference between a NamedList and a  SumpleOrderedMap. It 
> doesn't help to have both of them when they are doing exactly free same things



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

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to