[DISCUSS] Drop Heap Backend Synchronous snapshots

[Stephan Ewen]

Hi all!

I am wondering if there is any case for retaining the option to make
synchronous snapshots on the heap statebackend. Is anyone using that? Or
could we clean that code up and remove it?

Best,
Stephan

[Yun Tang]

+1 from my side for I did not see any real benefits if using synchronous snapshots.

Moreover, I think we should also remove the support of synchronous snapshots in DefaultOpeatorStateBackend and deprecate the config state.backend.async

Best
Yun Tang

On 12/5/19, 8:06 PM, "Stephan Ewen" <[hidden email]> wrote:

    Hi all!
   
    I am wondering if there is any case for retaining the option to make
    synchronous snapshots on the heap statebackend. Is anyone using that? Or
    could we clean that code up and remove it?
   
    Best,
    Stephan
   

[Congxian Qiu]

+1 from my side.

Best,
Congxian


Yun Tang <[hidden email]> 于2019年12月6日周五 下午12:30写道：

> +1 from my side for I did not see any real benefits if using synchronous
> snapshots.
>
> Moreover, I think we should also remove the support of synchronous
> snapshots in DefaultOpeatorStateBackend and deprecate the config
> state.backend.async
>
> Best
> Yun Tang
>
> On 12/5/19, 8:06 PM, "Stephan Ewen" <[hidden email]> wrote:
>
>     Hi all!
>
>     I am wondering if there is any case for retaining the option to make
>     synchronous snapshots on the heap statebackend. Is anyone using that?
> Or
>     could we clean that code up and remove it?
>
>     Best,
>     Stephan
>
>
>

[Yu Li]

+1 from my side.

FWIW, shall we also include @user ML into this discussion?

Best Regards,
Yu


On Mon, 9 Dec 2019 at 15:11, Congxian Qiu <[hidden email]> wrote:

> +1 from my side.
>
> Best,
> Congxian
>
>
> Yun Tang <[hidden email]> 于2019年12月6日周五 下午12:30写道：
>
> > +1 from my side for I did not see any real benefits if using synchronous
> > snapshots.
> >
> > Moreover, I think we should also remove the support of synchronous
> > snapshots in DefaultOpeatorStateBackend and deprecate the config
> > state.backend.async
> >
> > Best
> > Yun Tang
> >
> > On 12/5/19, 8:06 PM, "Stephan Ewen" <[hidden email]> wrote:
> >
> >     Hi all!
> >
> >     I am wondering if there is any case for retaining the option to make
> >     synchronous snapshots on the heap statebackend. Is anyone using that?
> > Or
> >     could we clean that code up and remove it?
> >
> >     Best,
> >     Stephan
> >
> >
> >
>

[Stephan Ewen]

Sounds like something we could do in 1.11 then, as part of simplification /
cleanup

On Mon, Dec 9, 2019 at 11:18 AM Yu Li <[hidden email]> wrote:

> +1 from my side.
>
> FWIW, shall we also include @user ML into this discussion?
>
> Best Regards,
> Yu
>
>
> On Mon, 9 Dec 2019 at 15:11, Congxian Qiu <[hidden email]> wrote:
>
> > +1 from my side.
> >
> > Best,
> > Congxian
> >
> >
> > Yun Tang <[hidden email]> 于2019年12月6日周五 下午12:30写道：
> >
> > > +1 from my side for I did not see any real benefits if using
> synchronous
> > > snapshots.
> > >
> > > Moreover, I think we should also remove the support of synchronous
> > > snapshots in DefaultOpeatorStateBackend and deprecate the config
> > > state.backend.async
> > >
> > > Best
> > > Yun Tang
> > >
> > > On 12/5/19, 8:06 PM, "Stephan Ewen" <[hidden email]> wrote:
> > >
> > >     Hi all!
> > >
> > >     I am wondering if there is any case for retaining the option to
> make
> > >     synchronous snapshots on the heap statebackend. Is anyone using
> that?
> > > Or
> > >     could we clean that code up and remove it?
> > >
> > >     Best,
> > >     Stephan
> > >
> > >
> > >
> >
>