[RESULT][VOTE] FLIP-437: Support ML Models in Flink SQL

[Hao Li]

Hi Dev,

I'm happy to announce that FLIP-437: Support ML Models in Flink SQL [1] has
been accepted with 7 approving votes (6 binding) [2]

Timo Walther (binding)
Jark Wu (binding)
Yu Chen (non-binding)
Piotr Nowojski (binding)
Leonard Xu (binding)
Martijn Visser (binding)
David Moravek (binding)

Thanks,
Hao

[1]
https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
[2] https://lists.apache.org/thread/gw1hfnqb05mwrstbtw43yh5tllrscgn6

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Hao Li]

Hi Dev,

Thanks all for voting. I'm closing the vote and the result will be posted
in a separate email.

Thanks,
Hao

On Wed, Apr 3, 2024 at 10:24 AM Hao Li  wrote:

> Thanks David Radley and David Moravek for the comments. I'll reply in the
> discussion thread.
>
> Hao
>
> On Wed, Apr 3, 2024 at 5:45 AM David Morávek  wrote:
>
>> +1 (binding)
>>
>> My only suggestion would be to move Catalog changes into a separate
>> interface to allow us to begin with lower stability guarantees. Existing
>> Catalogs would be able to opt-in by implementing it. It's a minor thing
>> though, overall the FLIP is solid and the direction is pretty exciting.
>>
>> Best,
>> D.
>>
>> On Wed, Apr 3, 2024 at 2:31 AM David Radley 
>> wrote:
>>
>> > Hi Hao,
>> > I don’t think this counts as an objection, I have some comments. I
>> should
>> > have put this on the discussion thread earlier but have just got to
>> this.
>> > - I suggest we can put a model version in the model resource. Versions
>> are
>> > notoriously difficult to add later; I don’t think we want to proliferate
>> > differently named models as a model mutates. We may want to work with
>> > non-latest models.
>> > - I see that the model name is the unique identifier. I realise this
>> would
>> > move away from the Oracle syntax – so may not be feasible short term;
>> but I
>> > wonder if we can have:
>> >  - a uuid as the main identifier and the model name as an attribute.
>> > or
>> >  - a namespace (or something like a system of origin)
>> > to help organise models with the same name.
>> > - does the model have an owner? I assume that Flink model resource is
>> the
>> > master of the model? I imagine in the future that a model that comes in
>> via
>> > a new connector could be kept up to date with the external model and
>> would
>> > not be allowed to be changed by anything other than the connector.
>> >
>> >Kind regards, David.
>> >
>> > From: Hao Li 
>> > Date: Friday, 29 March 2024 at 16:30
>> > To: dev@flink.apache.org 
>> > Subject: [EXTERNAL] [VOTE] FLIP-437: Support ML Models in Flink SQL
>> > Hi devs,
>> >
>> > I'd like to start a vote on the FLIP-437: Support ML Models in Flink
>> > SQL [1]. The discussion thread is here [2].
>> >
>> > The vote will be open for at least 72 hours unless there is an
>> objection or
>> > insufficient votes.
>> >
>> > [1]
>> >
>> >
>> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
>> >
>> > [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
>> >
>> > Thanks,
>> > Hao
>> >
>> > Unless otherwise stated above:
>> >
>> > IBM United Kingdom Limited
>> > Registered in England and Wales with number 741598
>> > Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU
>> >
>>
>

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Hao Li]

Thanks David Radley and David Moravek for the comments. I'll reply in the
discussion thread.

Hao

On Wed, Apr 3, 2024 at 5:45 AM David Morávek  wrote:

> +1 (binding)
>
> My only suggestion would be to move Catalog changes into a separate
> interface to allow us to begin with lower stability guarantees. Existing
> Catalogs would be able to opt-in by implementing it. It's a minor thing
> though, overall the FLIP is solid and the direction is pretty exciting.
>
> Best,
> D.
>
> On Wed, Apr 3, 2024 at 2:31 AM David Radley 
> wrote:
>
> > Hi Hao,
> > I don’t think this counts as an objection, I have some comments. I should
> > have put this on the discussion thread earlier but have just got to this.
> > - I suggest we can put a model version in the model resource. Versions
> are
> > notoriously difficult to add later; I don’t think we want to proliferate
> > differently named models as a model mutates. We may want to work with
> > non-latest models.
> > - I see that the model name is the unique identifier. I realise this
> would
> > move away from the Oracle syntax – so may not be feasible short term;
> but I
> > wonder if we can have:
> >  - a uuid as the main identifier and the model name as an attribute.
> > or
> >  - a namespace (or something like a system of origin)
> > to help organise models with the same name.
> > - does the model have an owner? I assume that Flink model resource is the
> > master of the model? I imagine in the future that a model that comes in
> via
> > a new connector could be kept up to date with the external model and
> would
> > not be allowed to be changed by anything other than the connector.
> >
> >    Kind regards, David.
> >
> > From: Hao Li 
> > Date: Friday, 29 March 2024 at 16:30
> > To: dev@flink.apache.org 
> > Subject: [EXTERNAL] [VOTE] FLIP-437: Support ML Models in Flink SQL
> > Hi devs,
> >
> > I'd like to start a vote on the FLIP-437: Support ML Models in Flink
> > SQL [1]. The discussion thread is here [2].
> >
> > The vote will be open for at least 72 hours unless there is an objection
> or
> > insufficient votes.
> >
> > [1]
> >
> >
> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
> >
> > [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
> >
> > Thanks,
> > Hao
> >
> > Unless otherwise stated above:
> >
> > IBM United Kingdom Limited
> > Registered in England and Wales with number 741598
> > Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU
> >
>

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[David Morávek]

+1 (binding)

My only suggestion would be to move Catalog changes into a separate
interface to allow us to begin with lower stability guarantees. Existing
Catalogs would be able to opt-in by implementing it. It's a minor thing
though, overall the FLIP is solid and the direction is pretty exciting.

Best,
D.

On Wed, Apr 3, 2024 at 2:31 AM David Radley  wrote:

> Hi Hao,
> I don’t think this counts as an objection, I have some comments. I should
> have put this on the discussion thread earlier but have just got to this.
> - I suggest we can put a model version in the model resource. Versions are
> notoriously difficult to add later; I don’t think we want to proliferate
> differently named models as a model mutates. We may want to work with
> non-latest models.
> - I see that the model name is the unique identifier. I realise this would
> move away from the Oracle syntax – so may not be feasible short term; but I
> wonder if we can have:
>  - a uuid as the main identifier and the model name as an attribute.
> or
>  - a namespace (or something like a system of origin)
> to help organise models with the same name.
> - does the model have an owner? I assume that Flink model resource is the
> master of the model? I imagine in the future that a model that comes in via
> a new connector could be kept up to date with the external model and would
> not be allowed to be changed by anything other than the connector.
>
>Kind regards, David.
>
> From: Hao Li 
> Date: Friday, 29 March 2024 at 16:30
> To: dev@flink.apache.org 
> Subject: [EXTERNAL] [VOTE] FLIP-437: Support ML Models in Flink SQL
> Hi devs,
>
> I'd like to start a vote on the FLIP-437: Support ML Models in Flink
> SQL [1]. The discussion thread is here [2].
>
> The vote will be open for at least 72 hours unless there is an objection or
> insufficient votes.
>
> [1]
>
> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
>
> [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
>
> Thanks,
> Hao
>
> Unless otherwise stated above:
>
> IBM United Kingdom Limited
> Registered in England and Wales with number 741598
> Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU
>

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[David Radley]

Hi Hao,
I don’t think this counts as an objection, I have some comments. I should have 
put this on the discussion thread earlier but have just got to this.
- I suggest we can put a model version in the model resource. Versions are 
notoriously difficult to add later; I don’t think we want to proliferate 
differently named models as a model mutates. We may want to work with 
non-latest models.
- I see that the model name is the unique identifier. I realise this would move 
away from the Oracle syntax – so may not be feasible short term; but I wonder 
if we can have:
 - a uuid as the main identifier and the model name as an attribute.
or
 - a namespace (or something like a system of origin)
to help organise models with the same name.
- does the model have an owner? I assume that Flink model resource is the 
master of the model? I imagine in the future that a model that comes in via a 
new connector could be kept up to date with the external model and would not be 
allowed to be changed by anything other than the connector.

   Kind regards, David.

From: Hao Li 
Date: Friday, 29 March 2024 at 16:30
To: dev@flink.apache.org 
Subject: [EXTERNAL] [VOTE] FLIP-437: Support ML Models in Flink SQL
Hi devs,

I'd like to start a vote on the FLIP-437: Support ML Models in Flink
SQL [1]. The discussion thread is here [2].

The vote will be open for at least 72 hours unless there is an objection or
insufficient votes.

[1]
https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL

[2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn

Thanks,
Hao

Unless otherwise stated above:

IBM United Kingdom Limited
Registered in England and Wales with number 741598
Registered office: PO Box 41, North Harbour, Portsmouth, Hants. PO6 3AU

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Martijn Visser]

+1 (binding)

On Wed, Apr 3, 2024 at 9:52 AM Leonard Xu  wrote:

> +1(binding)
>
> Best,
> Leonard
>
> > 2024年4月3日 下午3:37，Piotr Nowojski  写道：
> >
> > +1 (binding)
> >
> > Best,
> > Piotrek
> >
> > śr., 3 kwi 2024 o 04:29 Yu Chen  napisał(a):
> >
> >> +1 (non-binding)
> >>
> >> Looking forward to this future.
> >>
> >> Thanks,
> >> Yu Chen
> >>
> >>> 2024年4月3日 10:23，Jark Wu  写道：
> >>>
> >>> +1 (binding)
> >>>
> >>> Best,
> >>> Jark
> >>>
> >>> On Tue, 2 Apr 2024 at 15:12, Timo Walther  wrote:
> >>>
>  +1 (binding)
> 
>  Thanks,
>  Timo
> 
>  On 29.03.24 17:30, Hao Li wrote:
> > Hi devs,
> >
> > I'd like to start a vote on the FLIP-437: Support ML Models in Flink
> > SQL [1]. The discussion thread is here [2].
> >
> > The vote will be open for at least 72 hours unless there is an
> >> objection
>  or
> > insufficient votes.
> >
> > [1]
> >
> 
> >>
> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
> >
> > [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
> >
> > Thanks,
> > Hao
> >
> 
> 
> >>
> >>
>
>

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Leonard Xu]

+1(binding)

Best,
Leonard

> 2024年4月3日 下午3:37，Piotr Nowojski  写道：
> 
> +1 (binding)
> 
> Best,
> Piotrek
> 
> śr., 3 kwi 2024 o 04:29 Yu Chen  napisał(a):
> 
>> +1 (non-binding)
>> 
>> Looking forward to this future.
>> 
>> Thanks,
>> Yu Chen
>> 
>>> 2024年4月3日 10:23，Jark Wu  写道：
>>> 
>>> +1 (binding)
>>> 
>>> Best,
>>> Jark
>>> 
>>> On Tue, 2 Apr 2024 at 15:12, Timo Walther  wrote:
>>> 
 +1 (binding)
 
 Thanks,
 Timo
 
 On 29.03.24 17:30, Hao Li wrote:
> Hi devs,
> 
> I'd like to start a vote on the FLIP-437: Support ML Models in Flink
> SQL [1]. The discussion thread is here [2].
> 
> The vote will be open for at least 72 hours unless there is an
>> objection
 or
> insufficient votes.
> 
> [1]
> 
 
>> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
> 
> [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
> 
> Thanks,
> Hao
> 
 
 
>> 
>> 

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Piotr Nowojski]

+1 (binding)

Best,
Piotrek

śr., 3 kwi 2024 o 04:29 Yu Chen  napisał(a):

> +1 (non-binding)
>
> Looking forward to this future.
>
> Thanks,
> Yu Chen
>
> > 2024年4月3日 10:23，Jark Wu  写道：
> >
> > +1 (binding)
> >
> > Best,
> > Jark
> >
> > On Tue, 2 Apr 2024 at 15:12, Timo Walther  wrote:
> >
> >> +1 (binding)
> >>
> >> Thanks,
> >> Timo
> >>
> >> On 29.03.24 17:30, Hao Li wrote:
> >>> Hi devs,
> >>>
> >>> I'd like to start a vote on the FLIP-437: Support ML Models in Flink
> >>> SQL [1]. The discussion thread is here [2].
> >>>
> >>> The vote will be open for at least 72 hours unless there is an
> objection
> >> or
> >>> insufficient votes.
> >>>
> >>> [1]
> >>>
> >>
> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
> >>>
> >>> [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
> >>>
> >>> Thanks,
> >>> Hao
> >>>
> >>
> >>
>
>

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Yu Chen]

+1 (non-binding)

Looking forward to this future.

Thanks,
Yu Chen 

> 2024年4月3日 10:23，Jark Wu  写道：
> 
> +1 (binding)
> 
> Best,
> Jark
> 
> On Tue, 2 Apr 2024 at 15:12, Timo Walther  wrote:
> 
>> +1 (binding)
>> 
>> Thanks,
>> Timo
>> 
>> On 29.03.24 17:30, Hao Li wrote:
>>> Hi devs,
>>> 
>>> I'd like to start a vote on the FLIP-437: Support ML Models in Flink
>>> SQL [1]. The discussion thread is here [2].
>>> 
>>> The vote will be open for at least 72 hours unless there is an objection
>> or
>>> insufficient votes.
>>> 
>>> [1]
>>> 
>> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
>>> 
>>> [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
>>> 
>>> Thanks,
>>> Hao
>>> 
>> 
>> 

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Jark Wu]

+1 (binding)

Best,
Jark

On Tue, 2 Apr 2024 at 15:12, Timo Walther  wrote:

> +1 (binding)
>
> Thanks,
> Timo
>
> On 29.03.24 17:30, Hao Li wrote:
> > Hi devs,
> >
> > I'd like to start a vote on the FLIP-437: Support ML Models in Flink
> > SQL [1]. The discussion thread is here [2].
> >
> > The vote will be open for at least 72 hours unless there is an objection
> or
> > insufficient votes.
> >
> > [1]
> >
> https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL
> >
> > [2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn
> >
> > Thanks,
> > Hao
> >
>
>

Re: [VOTE] FLIP-437: Support ML Models in Flink SQL

[Timo Walther]

+1 (binding)

Thanks,
Timo

On 29.03.24 17:30, Hao Li wrote:

Hi devs,

I'd like to start a vote on the FLIP-437: Support ML Models in Flink
SQL [1]. The discussion thread is here [2].

The vote will be open for at least 72 hours unless there is an objection or
insufficient votes.

[1]
https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL

[2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn

Thanks,
Hao

[VOTE] FLIP-437: Support ML Models in Flink SQL

[Hao Li]

Hi devs,

I'd like to start a vote on the FLIP-437: Support ML Models in Flink
SQL [1]. The discussion thread is here [2].

The vote will be open for at least 72 hours unless there is an objection or
insufficient votes.

[1]
https://cwiki.apache.org/confluence/display/FLINK/FLIP-437%3A+Support+ML+Models+in+Flink+SQL

[2] https://lists.apache.org/thread/9z94m2bv4w265xb5l2mrnh4lf9m28ccn

Thanks,
Hao