[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    [jira] [Commented] (COLLECTIONS-439) TreeBag with comparator does not store non-key duplicates.
From:       "Thomas Neidhart (JIRA)" <jira () apache ! org>
Date:       2013-02-28 22:09:14
Message-ID: JIRA.12625707.1357148381022.361035.1362089354217 () arcas
[Download RAW message or body]


    [ https://issues.apache.org/jira/browse/COLLECTIONS-439?page=com.atlassian.jira.pl \
ugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13589979#comment-13589979 \
] 

Thomas Neidhart commented on COLLECTIONS-439:
---------------------------------------------

Ok, the implementation is pretty fine, but I am not yet fully convinced with the \
name.

Is it clear what a KeySortedBag is supposed to do when just looking at the name?
Does anybody have a better suggestion, or is this name acceptable?
                
> TreeBag with comparator does not store non-key duplicates.
> ----------------------------------------------------------
> 
> Key: COLLECTIONS-439
> URL: https://issues.apache.org/jira/browse/COLLECTIONS-439
> Project: Commons Collections
> Issue Type: Improvement
> Components: Bag
> Affects Versions: 3.2.1
> Reporter: Claude Warren
> Attachments: COLLECTIONS-439.tar.gz, SortedBag.java
> 
> 
> When storing objects that are sorted by a Comparator, if the differences in the \
> objects are not in the comparator the same version of the object is returned.  I \
> expected that the sorted bag would work like a DB table with a non-unique key -- \
> ordered but duplicate only determined by Object.equals(). I have implemented this \
> type of bag using a TreeMap of List.  My implementation uses the Jena \
> ExtendedIterator to make building the iterator() and array() methods easier. Will \
> attach the code.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic