Remove or deprecate IgniteAsyncSupport?

classic Classic list List threaded Threaded
4 messages Options
Reply | Threaded
Open this post in threaded view
|

Remove or deprecate IgniteAsyncSupport?

Vladimir Ozerov
Folks,

As you know we reworked our async API and added paired asynchronous methods
instead of using IgntieAsyncSupport infrastructure. Now our API is clean
and straightforward.

However, we didn't remove IgntieAsyncSupport, but deprecated it as user
impact might be huge. Now I think that may be ... may be it is better to
remove it right now thus forcing users to migrate to better API quicker?

Thoughts?

Vladimir.
Reply | Threaded
Open this post in threaded view
|

Re: Remove or deprecate IgniteAsyncSupport?

dsetrakyan
What is this obsession with breaking stuff? Let's deprecated it, it is a
big change on API.

On Fri, Apr 14, 2017 at 7:04 AM, Vladimir Ozerov <[hidden email]>
wrote:

> Folks,
>
> As you know we reworked our async API and added paired asynchronous methods
> instead of using IgntieAsyncSupport infrastructure. Now our API is clean
> and straightforward.
>
> However, we didn't remove IgntieAsyncSupport, but deprecated it as user
> impact might be huge. Now I think that may be ... may be it is better to
> remove it right now thus forcing users to migrate to better API quicker?
>
> Thoughts?
>
> Vladimir.
>
Reply | Threaded
Open this post in threaded view
|

Re: Remove or deprecate IgniteAsyncSupport?

Sergi
That API always was a big mistake. I'm for removing it completely.

Sergi

2017-04-14 18:01 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:

> What is this obsession with breaking stuff? Let's deprecated it, it is a
> big change on API.
>
> On Fri, Apr 14, 2017 at 7:04 AM, Vladimir Ozerov <[hidden email]>
> wrote:
>
> > Folks,
> >
> > As you know we reworked our async API and added paired asynchronous
> methods
> > instead of using IgntieAsyncSupport infrastructure. Now our API is clean
> > and straightforward.
> >
> > However, we didn't remove IgntieAsyncSupport, but deprecated it as user
> > impact might be huge. Now I think that may be ... may be it is better to
> > remove it right now thus forcing users to migrate to better API quicker?
> >
> > Thoughts?
> >
> > Vladimir.
> >
>
Reply | Threaded
Open this post in threaded view
|

Re: Remove or deprecate IgniteAsyncSupport?

dsetrakyan
On Fri, Apr 14, 2017 at 8:42 AM, Sergi Vladykin <[hidden email]>
wrote:

> That API always was a big mistake. I'm for removing it completely.
>

Sergi, I share your sentiment for cleaning up. But we cannot allow
ourselves such radical changes given that we have a fairly large user base
already.

Deprecating it seems like a safer approach for now.


>
> Sergi
>
> 2017-04-14 18:01 GMT+03:00 Dmitriy Setrakyan <[hidden email]>:
>
> > What is this obsession with breaking stuff? Let's deprecated it, it is a
> > big change on API.
> >
> > On Fri, Apr 14, 2017 at 7:04 AM, Vladimir Ozerov <[hidden email]>
> > wrote:
> >
> > > Folks,
> > >
> > > As you know we reworked our async API and added paired asynchronous
> > methods
> > > instead of using IgntieAsyncSupport infrastructure. Now our API is
> clean
> > > and straightforward.
> > >
> > > However, we didn't remove IgntieAsyncSupport, but deprecated it as user
> > > impact might be huge. Now I think that may be ... may be it is better
> to
> > > remove it right now thus forcing users to migrate to better API
> quicker?
> > >
> > > Thoughts?
> > >
> > > Vladimir.
> > >
> >
>