Re: [DISCUSS] Configuration for local recovery

2018-05-14 Thread Stefan Richter
I think that is a good idea +1. > Am 11.05.2018 um 20:41 schrieb Stephan Ewen : > > Hi! > > The configuration option (in flink-conf.yaml) for local recovery is currently > an enumeration with the values "DISABLED" and "ENABLE_FILE_BASED". > > I would suggest to change that,

Re: [DISCUSS] Configuration for local recovery

2018-05-14 Thread Fabian Hueske
+1 for Stephan's proposal. 2018-05-14 8:22 GMT+02:00 Shuyi Chen : > +1 to the proposal. IMO, the current option "ENABLE_FILE_BASED" contains > too much implementation details and might confuse the simple users. Having > a simple on/off toggle for majority of the users and an

Re: [DISCUSS] Configuration for local recovery

2018-05-14 Thread Shuyi Chen
+1 to the proposal. IMO, the current option "ENABLE_FILE_BASED" contains too much implementation details and might confuse the simple users. Having a simple on/off toggle for majority of the users and an advanced option for the experts to do the tuning will definitely make better user experience.

Re: [DISCUSS] Configuration for local recovery

2018-05-13 Thread Till Rohrmann
I agree with Stephan that a simple on/off configuration option for local recovery would be easier to understand and gives more flexibility wrt future changes. Cheers, Till On Sun, May 13, 2018 at 4:00 PM, sihua zhou wrote: > +1 for @Stephan's proposal, it makes the out of