If you are sorting it, the order of insertion and keeping it in order isn't
useful I guess.
Also, one could also use a ordered Set like LinkedHashSet or TreeSet
(obviously slightly slower in operation).

Would you be open to making that change, but I think in terms of expected
behavior we'd have metadata to be Collection of unique objects.

---
Regards,
Saptarshi PURKAYASTHA

My Tech Blog:  http://sunnytalkstech.blogspot.com
You Live by CHOICE, Not by CHANCE


On 6 April 2013 15:34, Morten Olav Hansen <[email protected]> wrote:

>
> On Fri, Apr 5, 2013 at 1:28 AM, Saptarshi Purkayastha <[email protected]>wrote:
>
>> Is there a reason not to use a Set here instead?
>>
>
> I'm sure some of them can be made sets. But I think the main reason was
> that we wanted a stable order. For most of the types, this doesn't matter,
> but for organisation units, we are exporting according to level. Not sure
> how useful this is, since I'm currently also doing the sort before starting
> a OU import.
>
> This is also used for generating the API web-pages, so that all types are
> sorted on names. Which makes paging etc more stable.
>
> I guess you could also use a SortedSet for this.
>
> --
> Morten
>
_______________________________________________
Mailing list: https://launchpad.net/~dhis2-devs
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~dhis2-devs
More help   : https://help.launchpad.net/ListHelp

Reply via email to