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