Re: Remove or deprecate IgniteAsyncSupport?

2017-04-14 Thread Dmitriy Setrakyan
On Fri, Apr 14, 2017 at 8:42 AM, Sergi Vladykin 
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 :
>
> > 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 
> > 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.
> > >
> >
>


Re: Remove or deprecate IgniteAsyncSupport?

2017-04-14 Thread Sergi Vladykin
That API always was a big mistake. I'm for removing it completely.

Sergi

2017-04-14 18:01 GMT+03:00 Dmitriy Setrakyan :

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


Re: Remove or deprecate IgniteAsyncSupport?

2017-04-14 Thread Dmitriy Setrakyan
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 
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.
>


Remove or deprecate IgniteAsyncSupport?

2017-04-14 Thread 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.