propose removing mlmd from kfp#12147
Conversation
hbelmiro
left a comment
hbelmiro
left a comment
There was a problem hiding this comment.
I won't have time to go deep into it. I'm leaving minor comments.
HumairAK
force-pushed
the
mlmd-removal
branch
4 times, most recently
from
e3ca11d to
6a4bfdd
Compare
HumairAK
force-pushed
the
mlmd-removal
branch
3 times, most recently
from
26dc9b8 to
0466990
Compare
gmfrasca
left a comment
gmfrasca
left a comment
There was a problem hiding this comment.
minor grammatical and/or lexiconical updates but otherwise lgtm
|
/lgtm |
Signed-off-by: Humair Khan <HumairAK@users.noreply.github.com>
|
/lgtm May the force be with us all during the implementation. 😆 |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mprahl The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
| - Blocks MySQL upgrades beyond 8.x and PostgreSQL support | ||
| - MLMD project is in maintenance mode and no longer actively maintained, leading to stagnated development and bug fixes | ||
|
|
||
| Removing MLMD and implementing native metadata management will eliminate these pain points while maintaining full functionality, resulting in a simpler, more maintainable system. |
There was a problem hiding this comment.
To me, a minor downside is that the MLMD store is written in C++, and it's hard for me to wrap my head around it)
There was a problem hiding this comment.
yes we come across this sentiment a lot, you are not alone!
Signed-off-by: Humair Khan <HumairAK@users.noreply.github.com>
Signed-off-by: Humair Khan <HumairAK@users.noreply.github.com>
5 participants
Description of your changes:
Part of: #11760
Checklist: