[VOTE] FLIP-54: Evolve ConfigOption and Configuration

[Timo Walther-2]

Hi everyone,

thanks for the great feedback we have received for the draft of FLIP-54.
The discussion seems to have reached an agreement. Of course this
doesn't mean that we can't propose further improvements on
ConfigOption's and Flink configuration in general in the future. It is
just one step towards having a better unified configuration for the project.

Please vote for the following design document:

https://docs.google.com/document/d/1IQ7nwXqmhCy900t2vQLEL3N2HIdMg-JO8vTzo1BtyKU/edit#

I will convert it to a Wiki page afterwards.

Thanks,
Timo

[Jark Wu-2]

+1 to the FLIP.


Regards,
Jark

> 在 2019年8月27日，19:28，Timo Walther <[hidden email]> 写道：
>
> Hi everyone,
>
> thanks for the great feedback we have received for the draft of FLIP-54. The discussion seems to have reached an agreement. Of course this doesn't mean that we can't propose further improvements on ConfigOption's and Flink configuration in general in the future. It is just one step towards having a better unified configuration for the project.
>
> Please vote for the following design document:
>
> https://docs.google.com/document/d/1IQ7nwXqmhCy900t2vQLEL3N2HIdMg-JO8vTzo1BtyKU/edit#
>
> I will convert it to a Wiki page afterwards.
>
> Thanks,
> Timo
>

[dwysakowicz]

+1 to the FLIP

Also I think we should mention that the voting will last at least 72
hours as requested by the bylaws until 30 Aug 14:00 CEST. (Correct me if
I am wrong Timo)

On 27/08/2019 13:32, Jark Wu wrote:

> +1 to the FLIP.
>
>
> Regards,
> Jark
>
>> 在 2019年8月27日，19:28，Timo Walther <[hidden email]> 写道：
>>
>> Hi everyone,
>>
>> thanks for the great feedback we have received for the draft of FLIP-54. The discussion seems to have reached an agreement. Of course this doesn't mean that we can't propose further improvements on ConfigOption's and Flink configuration in general in the future. It is just one step towards having a better unified configuration for the project.
>>
>> Please vote for the following design document:
>>
>> https://docs.google.com/document/d/1IQ7nwXqmhCy900t2vQLEL3N2HIdMg-JO8vTzo1BtyKU/edit#
>>
>> I will convert it to a Wiki page afterwards.
>>
>> Thanks,
>> Timo
>>

[JingsongLee-2]

+1 non-binding

Best,
Jingsong Lee


------------------------------------------------------------------
From:Dawid Wysakowicz <[hidden email]>
Send Time:2019年8月27日(星期二) 13:55
To:dev <[hidden email]>; Timo Walther <[hidden email]>
Subject:Re: [VOTE] FLIP-54: Evolve ConfigOption and Configuration

+1 to the FLIP

Also I think we should mention that the voting will last at least 72
hours as requested by the bylaws until 30 Aug 14:00 CEST. (Correct me if
I am wrong Timo)

On 27/08/2019 13:32, Jark Wu wrote:

> +1 to the FLIP.
>
>
> Regards,
> Jark
>
>> 在 2019年8月27日，19:28，Timo Walther <[hidden email]> 写道：
>>
>> Hi everyone,
>>
>> thanks for the great feedback we have received for the draft of FLIP-54. The discussion seems to have reached an agreement. Of course this doesn't mean that we can't propose further improvements on ConfigOption's and Flink configuration in general in the future. It is just one step towards having a better unified configuration for the project.
>>
>> Please vote for the following design document:
>>
>> https://docs.google.com/document/d/1IQ7nwXqmhCy900t2vQLEL3N2HIdMg-JO8vTzo1BtyKU/edit#
>>
>> I will convert it to a Wiki page afterwards.
>>
>> Thanks,
>> Timo
>>

[dwysakowicz]

Actually I wanted to propose a slight change to the proposal. Therefore I want to change my vote to -1 for now.

I suggest to change the Configurable interface to ConfigurableFactory:

public interface ConfigurableFactory<T> {

    /**

    * Creates an instance from the given configuration.

    */

    T fromConfiguration(ConfigurationReader configuration);

    /**

    * Writes this instance to the given configuration.

    */

    void toConfiguration(T value, ConfigurationWriter configuration);

}

And the corresponding method in the builder to:

<T> TypedConfigOptionBuilder<T> configurableType(Class<? extends ConfigurableFactory<T>> clazz) {

   return new TypedConfigOptionBuilder<>(key, clazz);

}
This way we can keep the "configurable" objects immutable.

Best,

Dawid

On 27/08/2019 13:28, Timo Walther wrote:

Hi everyone,

thanks for the great feedback we have received for the draft of FLIP-54. The discussion seems to have reached an agreement. Of course this doesn't mean that we can't propose further improvements on ConfigOption's and Flink configuration in general in the future. It is just one step towards having a better unified configuration for the project.

Please vote for the following design document:

https://docs.google.com/document/d/1IQ7nwXqmhCy900t2vQLEL3N2HIdMg-JO8vTzo1BtyKU/edit#

I will convert it to a Wiki page afterwards.

Thanks,
Timo

[Timo Walther-2]

Hi Dawid,

I'm a big fan of immutability and your suggestion makes sense to me.
Furthermore, I thought about the class name clashes of ConfigOptionGroup
and the existing ConfigGroup annotation. Maybe we should come up with a
better name.

I will make some additional adjustments to the FLIP. I hereby cancel the
vote. Further discussions should go into the [DISCUSS] thread until a
new vote is started.

Thanks,
Timo


On 27.08.19 16:12, Dawid Wysakowicz wrote:

>
> Actually I wanted to propose a slight change to the proposal.
> Therefore I want to change my vote to -1 for now.
>
> I suggest to change the Configurable interface to ConfigurableFactory:
>
> public interface ConfigurableFactory<T> {
>
>
> /**
>
> * Creates an instance from the given configuration.
>
> */
>
>     TfromConfiguration(ConfigurationReader configuration);
>
>
> /**
>
> * Writes this instance to the given configuration.
>
> */
>
> void toConfiguration(T value, ConfigurationWriter configuration);
>
> }
>
> And the corresponding method in the builder to:
>
> <T> TypedConfigOptionBuilder<T> configurableType(Class<? extends
> ConfigurableFactory<T>> clazz) {
>
> return new TypedConfigOptionBuilder<>(key, clazz);
>
> }
> This way we can keep the "configurable" objects immutable.
>
> Best,
>
> Dawid
>
> On 27/08/2019 13:28, Timo Walther wrote:
>> Hi everyone,
>>
>> thanks for the great feedback we have received for the draft of
>> FLIP-54. The discussion seems to have reached an agreement. Of course
>> this doesn't mean that we can't propose further improvements on
>> ConfigOption's and Flink configuration in general in the future. It
>> is just one step towards having a better unified configuration for
>> the project.
>>
>> Please vote for the following design document:
>>
>> https://docs.google.com/document/d/1IQ7nwXqmhCy900t2vQLEL3N2HIdMg-JO8vTzo1BtyKU/edit# 
>>
>>
>> I will convert it to a Wiki page afterwards.
>>
>> Thanks,
>> Timo
>>
>>