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, for a few reasons:
+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 advanced option for
>
+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.
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 the box experience bet
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, for a few reasons:
- Having values like "ENABLE_FILE_BASED" breaks with the style of the
other config options. Ha