[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