Integrated: 8284036: Make ConcurrentHashMap.CollectionView a sealed hierarchy

Jaikiran Pai jpai at openjdk.java.net
Mon Apr 11 01:27:45 UTC 2022


On Mon, 4 Apr 2022 06:55:10 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this change which now marks `CollectionView` as a `sealed` class with only `EntrySetView`, `KeySetView` and  `ValuesView` as the sub-classes? This change corresponds to https://bugs.openjdk.java.net/browse/JDK-8284036.
> 
> A CSR has also been drafted for this change https://bugs.openjdk.java.net/browse/JDK-8284272. As noted in the CSR, marking this class as `sealed` and marking `KeySetView` as `final` shouldn't have any impact in general and also in context of serialization/de-serialization.
> 
> tier1, tier2, tier3 tests have been run successfully with this change.

This pull request has now been integrated.

Changeset: eb3ead96
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/eb3ead96b300b05ddc709a1aa9dd03f664539697
Stats:     3 lines in 1 file changed: 0 ins; 0 del; 3 mod

8284036: Make ConcurrentHashMap.CollectionView a sealed hierarchy

Reviewed-by: smarks

-------------

PR: https://git.openjdk.java.net/jdk/pull/8085


More information about the core-libs-dev mailing list