+1 Support for standard YAML format facilitates specification Jing Ge <j...@ververica.com.invalid> 于2023年9月22日周五 02:23写道：
> Hi Junrui, > > +1 for following the standard. Thanks for your effort! > > Best regards, > Jing > > On Thu, Sep 21, 2023 at 5:09 AM Junrui Lee <jrlee....@gmail.com> wrote: > > > Hi Jane, > > > > Thank you for your valuable feedback and suggestions. > > I agree with your point about differentiating between "flink-config.yaml" > > and "flink-conf.yaml" to determine the standard syntax at a glance. > > > > While I understand your suggestion of using "flink-conf-default.yaml" to > > represent the default YAML file for Flink 1.x, I have been considering > > the option of using "flink-configuration.yaml" as the file name for the > > new configuration file. > > This name "flink-configuration.yaml" provides a clear distinction between > > the new and old configuration files based on their names, and it does not > > introduce any additional semantics. Moreover, this name > > "flink-configuration.yaml" can continue to be used in future versions > > FLINK-2.0. > > > > WDYT? If we can reach a consensus on this, I will update the FLIP > > documentation > > accordingly. > > > > Best regards, > > Junrui > > > > Jane Chan <qingyue....@gmail.com> 于2023年9月20日周三 23:38写道： > > > > > Hi Junrui, > > > > > > Thanks for driving this FLIP. +1 for adoption of the standard YAML > > syntax. > > > I just have one minor suggestion. It's a little bit challenging to > > > differentiate between `flink-config.yaml` and `flink-conf.yaml` to > > > determine which one uses the standard syntax at a glance. How about > > > using `flink-conf-default.yaml` to represent the default yaml file for > > > Flink 1.x? > > > > > > Best, > > > Jane > > > > > > On Wed, Sep 20, 2023 at 11:06 AM Junrui Lee <jrlee....@gmail.com> > wrote: > > > > > > > Hi devs, > > > > > > > > I would like to start a discussion about FLIP-366: > > > > Support standard YAML for FLINK configuration[1] > > > > > > > > The current flink-conf.yaml parser in FLINK is not a standard YAML > > > parser, > > > > which has some shortcomings. > > > > Firstly, it does not support nested structure configuration items and > > > only > > > > supports key-value pairs, resulting in poor readability. Secondly, if > > the > > > > value is a collection type, such as a List or Map, users are required > > to > > > > write the value in a FLINK-specific pattern, which is inconvenient to > > > use. > > > > Additionally, the parser of FLINK has some differences in syntax > > compared > > > > to the standard YAML parser, such as the syntax for parsing comments > > and > > > > null values. These inconsistencies can cause confusion for users, as > > seen > > > > in FLINK-15358 and FLINK-32740. > > > > > > > > By supporting standard YAML, these issues can be resolved, and users > > can > > > > create a Flink configuration file using third-party tools and > leverage > > > > some advanced YAML features. Therefore, we propose to support > standard > > > > YAML for FLINK configuration. > > > > > > > > You can find more details in the FLIP-366[1]. Looking forward to your > > > > feedback. > > > > > > > > [1] > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-366%3A+Support+standard+YAML+for+FLINK+configuration > > > > > > > > Best, > > > > Junrui > > > > > > > > > > -- Best ConradJam
