Re: Discussion - Change Public API Before Initial Release
For closure this has been cherry-picked into support/1.13. Thanks, Jake > On May 11, 2020, at 10:47 AM, Dave Barnes wrote: > > 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 wrote: > >> +1 >> >> From: Jacob Barrett >> Sent: May 8, 2020 21:26 >> To: 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 >> >>
Re: Discussion - Change Public API Before Initial Release
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 wrote: > +1 > > From: Jacob Barrett > Sent: May 8, 2020 21:26 > To: 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 > >
Re: Discussion - Change Public API Before Initial Release
+1 From: Jacob Barrett Sent: May 8, 2020 21:26 To: 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
Re: Discussion - Change Public API Before Initial Release
+1 On Fri, May 8, 2020 at 11:03 PM Dick Cavender wrote: > +1 > > On Fri, May 8, 2020 at 7:59 PM Owen Nichols wrote: > > > +1 > > > > > On May 8, 2020, at 7:57 PM, Robert Houghton > > wrote: > > > > > > +1 > > > > > > On Fri, May 8, 2020, 18:26 Jacob Barrett wrote: > > > > > >> 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 > > >> > > >> > > > > >
Re: Discussion - Change Public API Before Initial Release
+1 On Fri, May 8, 2020 at 7:59 PM Owen Nichols wrote: > +1 > > > On May 8, 2020, at 7:57 PM, Robert Houghton > wrote: > > > > +1 > > > > On Fri, May 8, 2020, 18:26 Jacob Barrett wrote: > > > >> 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 > >> > >> > >
Re: Discussion - Change Public API Before Initial Release
+1 > On May 8, 2020, at 7:57 PM, Robert Houghton wrote: > > +1 > > On Fri, May 8, 2020, 18:26 Jacob Barrett wrote: > >> 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 >> >>
Re: Discussion - Change Public API Before Initial Release
+1 On Fri, May 8, 2020, 18:26 Jacob Barrett wrote: > 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 > >
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