Re: Discussion: Condition field in the CR status

2024-05-23 Thread Lajith Koova
id you have a look at this, > >Kind regards, David, > > > From: Lajith Koova > Date: Monday, 13 May 2024 at 10:29 > To: dev@flink.apache.org > Subject: [EXTERNAL] Re: Discussion: Condition field in the CR status > Thanks for the feedback Jeyhun. > > Reg

RE: Discussion: Condition field in the CR status

2024-05-13 Thread David Radley
: [EXTERNAL] Re: Discussion: Condition field in the CR status Thanks for the feedback Jeyhun. Regarding Will the usage of Conditions be enabled by default? Or will there be any disadvantages for Flink users . Yes the conditions will be enabled by default for the CR. You are right , when

Re: Discussion: Condition field in the CR status

2024-05-13 Thread Lajith Koova
Thanks for the feedback Jeyhun. Regarding Will the usage of Conditions be enabled by default? Or will there be any disadvantages for Flink users . Yes the conditions will be enabled by default for the CR. You are right , when there are multiple conditions of the same type, approach is we will

Re: Discussion: Condition field in the CR status

2024-05-13 Thread Lajith Koova
Thanks for the feedback Gyula. I have updated a couple of samples with proposed conditions. Thanks Lajith K On Fri, May 3, 2024 at 12:24 PM Gyula Fóra wrote: > Hi Lajith! > > Can you please include some examples in the document to help reviewers? > Just some examples with the status and the

Re: Discussion: Condition field in the CR status

2024-05-03 Thread Jeyhun Karimov
Hi Lajith, Thanks a lot for driving this FLIP. Please find my comments below: - I echo Gyula that including some examples and further explanations might ease reader's work. With the current version, the FLIP is a bit hard to follow. - Will the usage of Conditions be enabled by default? Or will

Re: Discussion: Condition field in the CR status

2024-05-03 Thread Gyula Fóra
Hi Lajith! Can you please include some examples in the document to help reviewers? Just some examples with the status and the proposed conditions. Cheers, Gyula On Wed, May 1, 2024 at 9:06 AM Lajith Koova wrote: > Hello, > > > Starting discussion thread here to discuss a proposal to add