Conversation
|
Forgive me lack of knowledge, but what is Helm? |
|
If I'm not mistaken, this is to deploy UC on a K8 cluster? @EladLeev - by any chance do you have any verification scripts outside the the connection test? |
|
Yes, it's for deploying UC on top of K8s. Just to note, I assumed that the org and image name are unitycatalog here, if that's not the case, it's better to change the defaults Thanks! 🙏 |
|
Thanks @EladLeev - will review as soon as I can - much appreciated! |
|
Unity Catalog Helm chart is a great initiative! In my opinion, the Helm chart implementation looks very basic in the current state. After a brief review I found some misconfigurations that are concerning even for the initial version of the chart:
|
* delete table feature * handle onSuccess callback, navigate, display notification * PR feedback * More pr feedback * change button type * tableFullName prop * memoize
* delete table feature * handle onSuccess callback, navigate, display notification * PR feedback * More pr feedback * change button type * tableFullName prop * memoize
* delete table feature * handle onSuccess callback, navigate, display notification * PR feedback * More pr feedback * change button type * tableFullName prop * memoize
|
@EladLeev , great initiative as i'm looking for the helm chart to quickly spin up in the k8s to test out UC. |
To my knowledge, H2 is not for production environments; it is mostly used in dev and testing environments. Do you know if UC will support other databases like PostgreSQL, MySQL, etc.? |
Looks like Unity Catalog 0.2 (in development at the moment) supports other databases as well through providing custom Hibernate configuration and adding jars files with JDBC drivers to UC classpath. |
|
Why isn't this merged yet? |
Thanks for the reminder @ion-elgreco - I think a bunch of folks are currently out for the holidays. Would you be up for reviewing this? |
|
@ion-elgreco Probably the author doesn’t have time or not interested in continuing the development. For instance, he didn't address the issues from my comment #45 (comment). I started working on alternative PR some time ago, but unfortunately I also had to change my focus. I'm going to get back to it during holidays and check if latest UC release made operational aspects more clear and convenient to support it in Helm chart. |
|
@dennyglee @dnskr I might be able to allocate some time to this after January next year for either reviewing or helping to push this over the finish line |
|
Same here - I can revisit it on January and push an update. There are some missing features and toggles from this PR. |
|
Thanks in advance @EladLeev and @ion-elgreco . Let's definitely plan on restarting the push in January. IF it makes sense, perhaps for one of our community sync's we can dive into this as well? |
|
Following |
|
Hey @EladLeev - would it be helpful if we used one of the next community meetups to specifically discuss the design/issues related to this? Feel free to respond here and/or slack me via the Unity Catalog Slack to discuss at your convenience. Much appreciated! |
|
I guess this will be replaced by #895 (?) |
Create a base Helm chart for Unity Catalog based on #18.
Let me know if the
repositoryor image name should be changed.PR Checklist
docsis updatedDescription of changes