If you are changing an external API to fix this issue then it will need to
be for v 2.0.0 (breaking API change) under semantic versioning.

On Sun, Mar 19, 2023 at 12:10 PM PJ Fanning <[email protected]> wrote:

> Thanks everyone.
>
> I've merged Sam's and Seetaramayya's changes to the sample.
>
> If we run into other issues like this, it might be worthwhile for us
> to include conversion functions between the 2 types of Pekko JAPI
> Functions.
>
> I recognise that it is not difficult for users to do their own
> conversion code but it might be tidier for us to provide the
> conversion functions.
>
> Could we deprecate org.apache.pekko.japi.Function and any functions that
> use it?
>
> Regards,
> PJ
>
> On Fri, 17 Mar 2023 at 14:07, Matthew Benedict de Detrich
> <[email protected]> wrote:
> >
> > From what I remember another reason behind these custom function
> interfaces
> > is that the Java interfaces don't allow throwing exceptions in the apply
> > whereas some parts of Akka/Pekko rely on this behaviour.
> >
> > On Fri, Mar 17, 2023 at 12:03 PM Arnout Engelen <[email protected]>
> wrote:
> >
> > > On Thu, Mar 16, 2023 at 11:36 PM PJ Fanning <[email protected]>
> wrote:
> > > > As part of https://github.com/apache/incubator-pekko-samples/pull/4,
> > > we've run into an issue where one API returns a
> > > org.apache.pekko.japi.function.Function<?> but we need to convert it
> to a
> > > org.apache.pekko.japi.Function<?>.
> > > >
> > > > Does anyone know how to convert from one type of Function to the
> other
> > > type of Function?
> > >
> > >
> > >
> https://github.com/sam-byng/incubator-pekko-samples/pull/2/commits/7e1c9b4dcaca632d844b32beda1f8b21055ae33a
> > >
> > > I haven't refreshed my memory on the details, but it is possible the
> > > only reason that there are two Function types there is because we
> > > wanted to change it, but had to introduce a second type to avoid
> > > breaking (binary?) compatibility in code already using the 'old'
> > > variation.
> > >
> > > Do we want to 'clean up' such situations in Pekko? Does that need to
> > > happen before our first release or can we keep some 'stabilization
> > > period' before we start promising binary compatibility?
> > >
> > >
> > > Kind regards,
> > >
> > > Arnout
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: [email protected]
> > > For additional commands, e-mail: [email protected]
> > >
> > >
> >
> > --
> >
> > Matthew de Detrich
> >
> > *Aiven Deutschland GmbH*
> >
> > Immanuelkirchstraße 26, 10405 Berlin
> >
> > Amtsgericht Charlottenburg, HRB 209739 B
> >
> > Geschäftsführer: Oskari Saarenmaa & Hannu Valtonen
> >
> > *m:* +491603708037
> >
> > *w:* aiven.io *e:* [email protected]
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
>

Reply via email to