+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 was not supposed to be used widely
(mostly for our internal stuff), and would make it clear that when in
doubt: use the JDK provided one.

Martijn


On Wed, May 6, 2020 at 12:36 PM Martin Grigorov <mgrigo...@apache.org>
wrote:

On Wed, May 6, 2020 at 1:20 PM Maxim Solodovnik <solomax...@gmail.com>
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,
Maxim


Reply via email to