Re: ConcurrentHashSet to be removed from wicket code base

2020-05-07 Thread Martin Grigorov
On Thu, May 7, 2020 at 3:02 PM Martijn Dashorst wrote: > IIRC Micromap was created for serialization and memory efficiency. Has that > changed? > Java 9+ has Map.of(k, v) method that uses java.util.ImmutableCollections.Map1 and it is quite optimized. I think the replacement is good! > > Martijn

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-07 Thread Maxim Solodovnik
Hello Martijn, On Thu, 7 May 2020 at 19:02, Martijn Dashorst wrote: > IIRC Micromap was created for serialization and memory efficiency. Has that > changed? > >From what I can see using "so to declaration" in JDK11 Map.of(key, value) returns static final class Map1 extends AbstractImmutabl

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-07 Thread Martijn Dashorst
IIRC Micromap was created for serialization and memory efficiency. Has that changed? Martijn On Wed, May 6, 2020 at 2:18 PM Maxim Solodovnik wrote: > MicroMap can easily be replaced with Map.of(key, value) > > I can deprecate it for wicket9 in same PR, should I? > > On Wed, 6 May 2020 at 18:06,

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-06 Thread Maxim Solodovnik
MicroMap can easily be replaced with Map.of(key, value) I can deprecate it for wicket9 in same PR, should I? On Wed, 6 May 2020 at 18:06, Maxim Solodovnik wrote: > I would propose to remove it in Wicket9, but don't want to slow down the > release :) > Will create PR :) > > On Wed, 6 May 2020 at

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-06 Thread Maxim Solodovnik
I would propose to remove it in Wicket9, but don't want to slow down the release :) Will create PR :) On Wed, 6 May 2020 at 17:56, Andrea Del Bene wrote: > +1 to replace it with ConcurrentHashMap.newKeySet() > > On 06/05/20 12:53, Martijn Dashorst wrote: > > I'd go further and deprecate it in wi

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-06 Thread Andrea Del Bene
+1 to replace it with ConcurrentHashMap.newKeySet() On 06/05/20 12:53, Martijn Dashorst wrote: I'd go further and deprecate it in wicket 8 as well... Since the replacement is in Java 8. I would be +1 on removing it from Wicket 9 final if it came to a vote. It is not a core class in Wicket, it w

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-06 Thread Martijn Dashorst
I'd go further and deprecate it in wicket 8 as well... Since the replacement is in Java 8. I would be +1 on removing it from Wicket 9 final if it came to a vote. It is not a core class in Wicket, it was not supposed to be used widely (mostly for our internal stuff), and would make it clear that wh

Re: ConcurrentHashSet to be removed from wicket code base

2020-05-06 Thread Martin Grigorov
On Wed, May 6, 2020 at 1:20 PM Maxim Solodovnik wrote: > Hello All, > > ConcurrentHashSet can be safely removed from wicket codebase due to > since Java8 it is possible to use ConcurrentHashMap.newKeySet() > > Can we @deprecate in in wicket9 and remove in Wicket10? > +1 > > -- > Best regards,