RFR: 8235139: Remove the socket impl factory mechanism [v3]

Daniel Fuchs dfuchs at openjdk.java.net
Fri Feb 5 11:35:42 UTC 2021


On Fri, 5 Feb 2021 11:25:57 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
>
> Patrick Concannon has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
> 
>  - 8235139: Removed @Deprecated annotation from factory fields
>  - Merge remote-tracking branch 'origin/master' into JDK-8235139
>  - 8235139: Updated @deprecated text for setSocketFactory methods in Socket and ServerSocket
>  - Merge remote-tracking branch 'origin/master' into JDK-8235139
>  - 8235139: Remove the socket impl factory mechanism

Marked as reviewed by dfuchs (Reviewer).

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

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


More information about the net-dev mailing list