RFR: 8235139: Remove the socket impl factory mechanism

[Alan Bateman]

On Wed, 3 Feb 2021 11:03:51 GMT, Patrick Concannon <pconcannon at openjdk.org> wrote:

> Hi,
> 
> Could someone please review my proposed changeset for JDK-8235139: '`Remove the socket impl factory mechanism`' ?
> 
> These changes propose to deprecate (for the eventual removal) the API points for statically configuring a system-wide factory for the `Socket`, `ServerSocket`, and `DatagramSocket` types in the `java.net package`. Specifically, the following:
> 
> **Methods**:
> - `static void Socket.setSocketImplFactory​(SocketImplFactory fac)`
> - `static void ServerSocket.setSocketFactory​(SocketImplFactory fac)`
> - `static void DatagramSocket.setDatagramSocketImplFactory​(DatagramSocketImplFactory fac)`
> 
> **Types**:
> - `java.net SocketImplFactory`
> - `java.net DatagramSocketImplFactory`
> 
> The CSR for[ JDK-8220494](https://bugs.openjdk.java.net/browse/JDK-8220494) contains some verbiage about the potential
> issues that setting factories can have, and alludes to their possible
> future removal. 
> 
> Kind regards,
> Patrick

src/java.base/share/classes/java/net/DatagramSocket.java line 959:

> 957:      *    and using the {@linkplain #DatagramSocket(DatagramSocketImpl) protected
> 958:      *    constructor} that takes an {@linkplain DatagramSocketImpl implementation}
> 959:      *    as a parameter.

This wording is good, are you planning to add this to the deprecated methods in Server and ServerSocket too?

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

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