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 <[email protected]>
Date: Friday, 29 March 2024 at 16:30
To: [email protected] <[email protected]>
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
