ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy Setrakyan <dsetrak...@apache.org>
Subject Re: ConcurrentLinkedHashMap works incorrectly after clear()
Date Thu, 09 Aug 2018 15:05:14 GMT
On Thu, Aug 9, 2018 at 8:16 AM, Alexey Goncharuk <alexey.goncharuk@gmail.com
> wrote:

> Guys, I think we can definitely change current implementation of CLHM with
> a more stable one, but as a temporal solution I see nothing wrong with
> throwing an UnsupportedOperationException from clear() method. Ilya already
> provided a patch which replaces all clear() calls with a new map creation,
> semantically it has the same behavior as a correct clear() method.
> I suggest to merge the provided PR because IgniteH2Indexing is broken
> because of this bug and then create a ticket to replace/fix/whatever it
> feels right to do with current CLHM.
> Thoughts?

I agree. This sounds like a less risky approach.


  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message