search

DaveAKing / guava-libraries

Automatically exported from code.google.com/p/guava-libraries
Apache License 2.0
0 stars 0 forks source link

containsValue not implemented in ImmutableBiMap #1416

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
In version 14.0.1 of Guava ImmutableBiMap containsValue method is not 
overridden and points to ImmutableMap containsValue which results in 
significant delays. It was implemented properly in 13.0.1.

Original issue reported on code.google.com by ivan.nik...@cognitive-security.com on 16 May 2013 at 12:07

GoogleCodeExporter commented 9 years ago

Original comment by kurt.kluever on 16 May 2013 at 1:40

GoogleCodeExporter commented 9 years ago 
Fixed internally; will be mirrored out in the next few days.  We'll probably at 
least want to discuss a point release.

Original comment by lowas...@google.com on 16 May 2013 at 8:48

GoogleCodeExporter commented 9 years ago 
Hmm, I don't know how I feel about a point release for this. It's unfortunate 
that the performance of containsValue() is not what it could be without the 
fix, but A) the performance is not worse than what you get with a normal map 
and B) there's a workaround (inverse().containsKey(value)).

Original comment by cgdec...@gmail.com on 16 May 2013 at 9:20

GoogleCodeExporter commented 9 years ago 
-1 on a point release...the 15 RC will be hitting the shelves in a couple weeks 
anyways.

Original comment by kak@google.com on 16 May 2013 at 10:57

GoogleCodeExporter commented 9 years ago 
This issue has been migrated to GitHub.

It can be found at https://github.com/google/guava/issues/<issue id>

Original comment by cgdecker@google.com on 1 Nov 2014 at 4:12

GoogleCodeExporter commented 9 years ago

Original comment by cgdecker@google.com on 3 Nov 2014 at 9:08