+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" <se...@apache.org> 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