[RESULT][VOTE] FLIP-405: Migrate string configuration key to ConfigOption
Hi devs, I'm happy to announce that FLIP-405: Migrate string configuration key to ConfigOption [1] has been accepted with 4 approving votes (3 binding) [2]: - Rui Fan (binding) - Hang Ruan (non-binding) - Xintong Song (binding) - Zhu Zhu (binding) There are no disapproving votes. Thanks again to everyone who participated in the discussion and voting. Best regards, Xuannan [1] https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption [2] https://lists.apache.org/thread/joyr7bxpo0lcj1zfzdj5nv0lrhb303rx
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
Hi all, Thank you all! Closing the vote. The result will be announced in a separate email. Best regards, Xuannan On Fri, Jan 12, 2024 at 2:44 PM Zhu Zhu wrote: > > +1 (binding) > > Thanks, > Zhu > > Xuannan Su 于2024年1月12日周五 14:24写道： > > > Hi all, > > > > I would like to clarify the statement regarding the first improvement > > from the previous email, as it was incomplete. To be more specific, we > > will also deprecate the getClass(String key, Class > > defaultValue, ClassLoader classLoader) and setClass(String key, > > Class klazz), as they are intended for internal use only and are no > > longer in use. > > > > To summarize, the first improvement should be as follows: > > > > We will mark the getBytes(String key, byte[] defaultValue) and > > setBytes(String key, byte[] bytes) methods as @Internal, as they are > > intended for internal use only. Additionally, we will > > deprecatesgetClass(String key, Class defaultValue, > > ClassLoader classLoader) and setClass(String key, Class klazz) > > methods, as they are intended for internal use only and are no longer > > in use. > > > > I apologize for any confusion caused by the incomplete information. > > > > Best regards, > > Xuannan > > > > On Fri, Jan 12, 2024 at 1:36 PM Xuannan Su wrote: > > > > > > Hi all, > > > > > > During voting, we identified two improvements we'd like to make to the > > FLIP: > > > > > > - We will mark the getBytes(String key, byte[] defaultValue) and > > > setBytes(String key, byte[] bytes) methods as @Internal, as they are > > > intended for internal use only. > > > - In addition to marking all getXxx(ConfigOption configOption) methods > > > as @Deprecated, we will also mark the getXxx(ConfigOption > > > configOption, Xxx overrideDefault) methods as @Deprecated. These will > > > be replaced with T get(ConfigOption configOption, T overrideDefault). > > > > > > We had an offline discussion with all the voters and reached a > > > consensus on the above changes. Therefore, we will not initiate > > > another voting thread unless there are objections. > > > > > > Best regards, > > > Xuannan > > > > > > > > > On Tue, Jan 9, 2024 at 11:43 AM Xintong Song > > wrote: > > > > > > > > +1 (binding) > > > > > > > > Best, > > > > > > > > Xintong > > > > > > > > > > > > > > > > On Mon, Jan 8, 2024 at 1:48 PM Hang Ruan > > wrote: > > > > > > > > > +1(non-binding) > > > > > > > > > > Best, > > > > > Hang > > > > > > > > > > Rui Fan <1996fan...@gmail.com> 于2024年1月8日周一 13:04写道： > > > > > > > > > > > +1(binding) > > > > > > > > > > > > Best, > > > > > > Rui > > > > > > > > > > > > On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su > > wrote: > > > > > > > > > > > > > Hi everyone, > > > > > > > > > > > > > > Thanks for all the feedback about the FLIP-405: Migrate string > > > > > > > configuration key to ConfigOption [1] [2]. > > > > > > > > > > > > > > I'd like to start a vote for it. The vote will be open for at > > least 72 > > > > > > > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there > > is an > > > > > > > objection or an insufficient number of votes. > > > > > > > > > > > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > > > > > > > [2] > > https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > > > > > > > > > > > > > > > > > > > Best, > > > > > > > Xuannan > > > > > > > > > > > > > > > > > > > >
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
+1 (binding) Thanks, Zhu Xuannan Su 于2024年1月12日周五 14:24写道： > Hi all, > > I would like to clarify the statement regarding the first improvement > from the previous email, as it was incomplete. To be more specific, we > will also deprecate the getClass(String key, Class > defaultValue, ClassLoader classLoader) and setClass(String key, > Class klazz), as they are intended for internal use only and are no > longer in use. > > To summarize, the first improvement should be as follows: > > We will mark the getBytes(String key, byte[] defaultValue) and > setBytes(String key, byte[] bytes) methods as @Internal, as they are > intended for internal use only. Additionally, we will > deprecatesgetClass(String key, Class defaultValue, > ClassLoader classLoader) and setClass(String key, Class klazz) > methods, as they are intended for internal use only and are no longer > in use. > > I apologize for any confusion caused by the incomplete information. > > Best regards, > Xuannan > > On Fri, Jan 12, 2024 at 1:36 PM Xuannan Su wrote: > > > > Hi all, > > > > During voting, we identified two improvements we'd like to make to the > FLIP: > > > > - We will mark the getBytes(String key, byte[] defaultValue) and > > setBytes(String key, byte[] bytes) methods as @Internal, as they are > > intended for internal use only. > > - In addition to marking all getXxx(ConfigOption configOption) methods > > as @Deprecated, we will also mark the getXxx(ConfigOption > > configOption, Xxx overrideDefault) methods as @Deprecated. These will > > be replaced with T get(ConfigOption configOption, T overrideDefault). > > > > We had an offline discussion with all the voters and reached a > > consensus on the above changes. Therefore, we will not initiate > > another voting thread unless there are objections. > > > > Best regards, > > Xuannan > > > > > > On Tue, Jan 9, 2024 at 11:43 AM Xintong Song > wrote: > > > > > > +1 (binding) > > > > > > Best, > > > > > > Xintong > > > > > > > > > > > > On Mon, Jan 8, 2024 at 1:48 PM Hang Ruan > wrote: > > > > > > > +1(non-binding) > > > > > > > > Best, > > > > Hang > > > > > > > > Rui Fan <1996fan...@gmail.com> 于2024年1月8日周一 13:04写道： > > > > > > > > > +1(binding) > > > > > > > > > > Best, > > > > > Rui > > > > > > > > > > On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su > wrote: > > > > > > > > > > > Hi everyone, > > > > > > > > > > > > Thanks for all the feedback about the FLIP-405: Migrate string > > > > > > configuration key to ConfigOption [1] [2]. > > > > > > > > > > > > I'd like to start a vote for it. The vote will be open for at > least 72 > > > > > > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there > is an > > > > > > objection or an insufficient number of votes. > > > > > > > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > > > > > > [2] > https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > > > > > > > > > > > > > > > > Best, > > > > > > Xuannan > > > > > > > > > > > > > > > >
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
Hi all, I would like to clarify the statement regarding the first improvement from the previous email, as it was incomplete. To be more specific, we will also deprecate the getClass(String key, Class defaultValue, ClassLoader classLoader) and setClass(String key, Class klazz), as they are intended for internal use only and are no longer in use. To summarize, the first improvement should be as follows: We will mark the getBytes(String key, byte[] defaultValue) and setBytes(String key, byte[] bytes) methods as @Internal, as they are intended for internal use only. Additionally, we will deprecatesgetClass(String key, Class defaultValue, ClassLoader classLoader) and setClass(String key, Class klazz) methods, as they are intended for internal use only and are no longer in use. I apologize for any confusion caused by the incomplete information. Best regards, Xuannan On Fri, Jan 12, 2024 at 1:36 PM Xuannan Su wrote: > > Hi all, > > During voting, we identified two improvements we'd like to make to the FLIP: > > - We will mark the getBytes(String key, byte[] defaultValue) and > setBytes(String key, byte[] bytes) methods as @Internal, as they are > intended for internal use only. > - In addition to marking all getXxx(ConfigOption configOption) methods > as @Deprecated, we will also mark the getXxx(ConfigOption > configOption, Xxx overrideDefault) methods as @Deprecated. These will > be replaced with T get(ConfigOption configOption, T overrideDefault). > > We had an offline discussion with all the voters and reached a > consensus on the above changes. Therefore, we will not initiate > another voting thread unless there are objections. > > Best regards, > Xuannan > > > On Tue, Jan 9, 2024 at 11:43 AM Xintong Song wrote: > > > > +1 (binding) > > > > Best, > > > > Xintong > > > > > > > > On Mon, Jan 8, 2024 at 1:48 PM Hang Ruan wrote: > > > > > +1(non-binding) > > > > > > Best, > > > Hang > > > > > > Rui Fan <1996fan...@gmail.com> 于2024年1月8日周一 13:04写道： > > > > > > > +1(binding) > > > > > > > > Best, > > > > Rui > > > > > > > > On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su wrote: > > > > > > > > > Hi everyone, > > > > > > > > > > Thanks for all the feedback about the FLIP-405: Migrate string > > > > > configuration key to ConfigOption [1] [2]. > > > > > > > > > > I'd like to start a vote for it. The vote will be open for at least 72 > > > > > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there is an > > > > > objection or an insufficient number of votes. > > > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > > > > > [2] https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > > > > > > > > > > > > > Best, > > > > > Xuannan > > > > > > > > > > > >
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
Hi all, During voting, we identified two improvements we'd like to make to the FLIP: - We will mark the getBytes(String key, byte[] defaultValue) and setBytes(String key, byte[] bytes) methods as @Internal, as they are intended for internal use only. - In addition to marking all getXxx(ConfigOption configOption) methods as @Deprecated, we will also mark the getXxx(ConfigOption configOption, Xxx overrideDefault) methods as @Deprecated. These will be replaced with T get(ConfigOption configOption, T overrideDefault). We had an offline discussion with all the voters and reached a consensus on the above changes. Therefore, we will not initiate another voting thread unless there are objections. Best regards, Xuannan On Tue, Jan 9, 2024 at 11:43 AM Xintong Song wrote: > > +1 (binding) > > Best, > > Xintong > > > > On Mon, Jan 8, 2024 at 1:48 PM Hang Ruan wrote: > > > +1(non-binding) > > > > Best, > > Hang > > > > Rui Fan <1996fan...@gmail.com> 于2024年1月8日周一 13:04写道： > > > > > +1(binding) > > > > > > Best, > > > Rui > > > > > > On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su wrote: > > > > > > > Hi everyone, > > > > > > > > Thanks for all the feedback about the FLIP-405: Migrate string > > > > configuration key to ConfigOption [1] [2]. > > > > > > > > I'd like to start a vote for it. The vote will be open for at least 72 > > > > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there is an > > > > objection or an insufficient number of votes. > > > > > > > > > > > > > > > > [1] > > > > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > > > > [2] https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > > > > > > > > > > Best, > > > > Xuannan > > > > > > > > >
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
+1 (binding) Best, Xintong On Mon, Jan 8, 2024 at 1:48 PM Hang Ruan wrote: > +1(non-binding) > > Best, > Hang > > Rui Fan <1996fan...@gmail.com> 于2024年1月8日周一 13:04写道： > > > +1(binding) > > > > Best, > > Rui > > > > On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su wrote: > > > > > Hi everyone, > > > > > > Thanks for all the feedback about the FLIP-405: Migrate string > > > configuration key to ConfigOption [1] [2]. > > > > > > I'd like to start a vote for it. The vote will be open for at least 72 > > > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there is an > > > objection or an insufficient number of votes. > > > > > > > > > > > > [1] > > > > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > > > [2] https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > > > > > > > Best, > > > Xuannan > > > > > >
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
+1(non-binding) Best, Hang Rui Fan <1996fan...@gmail.com> 于2024年1月8日周一 13:04写道： > +1(binding) > > Best, > Rui > > On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su wrote: > > > Hi everyone, > > > > Thanks for all the feedback about the FLIP-405: Migrate string > > configuration key to ConfigOption [1] [2]. > > > > I'd like to start a vote for it. The vote will be open for at least 72 > > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there is an > > objection or an insufficient number of votes. > > > > > > > > [1] > > > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > > [2] https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > > > > Best, > > Xuannan > > >
Re: [VOTE] FLIP-405: Migrate string configuration key to ConfigOption
+1(binding) Best, Rui On Mon, Jan 8, 2024 at 1:00 PM Xuannan Su wrote: > Hi everyone, > > Thanks for all the feedback about the FLIP-405: Migrate string > configuration key to ConfigOption [1] [2]. > > I'd like to start a vote for it. The vote will be open for at least 72 > hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there is an > objection or an insufficient number of votes. > > > > [1] > https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption > [2] https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 > > > Best, > Xuannan >
[VOTE] FLIP-405: Migrate string configuration key to ConfigOption
Hi everyone, Thanks for all the feedback about the FLIP-405: Migrate string configuration key to ConfigOption [1] [2]. I'd like to start a vote for it. The vote will be open for at least 72 hours(excluding weekends,until Jan 11, 12:00AM GMT) unless there is an objection or an insufficient number of votes. [1] https://cwiki.apache.org/confluence/display/FLINK/FLIP-405%3A+Migrate+string+configuration+key+to+ConfigOption [2] https://lists.apache.org/thread/zfw1b1g3679yn0ppjbsokfrsx9k7ybg0 Best, Xuannan
