RFR: 8253119: Remove the legacy PlainSocketImpl and PlainDatagramSocketImpl implementation

Patrick Concannon pconcannon at openjdk.java.net
Wed Jun 23 12:15:11 UTC 2021


Hi,

Could someone please review my changes for the removal of the legacy `PlainSocketImpl` and `PlainDatagramSocketImpl` implementations?

In JDK 13, JEP 353 provided a drop in replacement for the legacy `PlainSocketImpl` implementation. Since JDK 13, the `PlainSocketImpl` implementation was no longer used but included a mitigation mechanism to reduce compatibility risks in the form of a JDK-specific property `jdk.net.usePlainSocketImpl` allowing to switch back to the old implementation. 
Similarly, in JDK 15, JEP 373 provided a new implementation for `DatagramSocket` and `MulticastSocket`, with a JDK-specific property `jdk.net.usePlainDatagramSocketImpl` also allowing the user to switch back to the old implementation in case of compatibility issue.

As these implementations (and the mechanisms they use to enable them to mitigate compatibility issues) have been deemed no longer necessary, they now represent a maintenance burden. This patch looks at removing them from the JDK.

Kind regards,
Patrick

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

Commit messages:
 - 8253119: Remove the legacy PlainSocketImpl and PlainDatagramSocketImpl implementation

Changes: https://git.openjdk.java.net/jdk/pull/4574/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4574&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8253119
  Stats: 11046 lines in 78 files changed: 137 ins; 10836 del; 73 mod
  Patch: https://git.openjdk.java.net/jdk/pull/4574.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/4574/head:pull/4574

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


More information about the nio-dev mailing list