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 > >