On Thu, Aug 9, 2018 at 8:16 AM, Alexey Goncharuk <alexey.goncha...@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. D.