Plenty of votes, go ahead, Jake, and add this to the support/1.13 branch.

On Mon, May 11, 2020 at 8:36 AM Joris Melchior <jmelch...@vmware.com> wrote:

> +1
> ________________________________
> From: Jacob Barrett <jbarr...@pivotal.io>
> Sent: May 8, 2020 21:26
> To: dev@geode.apache.org <dev@geode.apache.org>
> Subject: Discussion - Change Public API Before Initial Release
>
> Hey Ya’ll,
>
> We have a new API going into 1.13 that has an inconsistency I want to
> address before we are stuck with it. The class SniSocketFactory should be
> renamed SniProxySocketFactory. The class in question produces objects of
> type SniProxySocket. An "SNI socket" isn’t a thing but an SNI proxy is a
> thing. The factory class that produces proxy socket factories (yes a meta
> factory) ProxySocketFactories uses the “ProxySocket” name as well. It fells
> very inconsistent with all the other classes that make up with API for
> SniSocketFactory to not have a proxy in it and be called
> SniProxySocketFactory.
>
> To be very clear, this API has not made it out of development yet but is
> in the 1.13 release branch. Can I get a few thumbs up to open an ticket and
> pick it into the 1.13 release branch before it goes out?
>
> Thanks,
> Jake
>
>

Reply via email to