On Wed, 3 Feb 2021 11:03:51 GMT, Patrick Concannon <pconcan...@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. > > The CSR for this issue can be viewed here: > https://bugs.openjdk.java.net/browse/JDK-8261228 > > Kind regards, > Patrick This pull request has now been integrated. Changeset: 16623734 Author: Patrick Concannon <pconcan...@openjdk.org> URL: https://git.openjdk.java.net/jdk/commit/16623734 Stats: 58 lines in 3 files changed: 31 ins; 22 del; 5 mod 8235139: Deprecate the socket impl factory mechanism Reviewed-by: dfuchs, alanb, chegar ------------- PR: https://git.openjdk.java.net/jdk/pull/2375