mchades merged PR #5848:
URL: https://github.com/apache/gravitino/pull/5848
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@gravitin
jerryshao commented on PR #5848:
URL: https://github.com/apache/gravitino/pull/5848#issuecomment-2547890860
> I noticed that we have the listModelVersions operation, but why don't we
have a listModelAlias method?
There's no specific reason why this "listModelAliases" API is not added,
jerryshao commented on code in PR #5848:
URL: https://github.com/apache/gravitino/pull/5848#discussion_r1888155923
##
catalogs/catalog-model/build.gradle.kts:
##
@@ -40,11 +40,17 @@ dependencies {
exclude(group = "*")
}
- compileOnly(libs.guava)
-
+ implementation(li
mchades commented on code in PR #5848:
URL: https://github.com/apache/gravitino/pull/5848#discussion_r1888020466
##
catalogs/catalog-model/build.gradle.kts:
##
@@ -40,11 +40,17 @@ dependencies {
exclude(group = "*")
}
- compileOnly(libs.guava)
-
+ implementation(libs
jerryshao commented on PR #5848:
URL: https://github.com/apache/gravitino/pull/5848#issuecomment-2545404545
@yuqi1129 @mchades @xloya can you please help to review this PR? Thanks.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
jerryshao opened a new pull request, #5848:
URL: https://github.com/apache/gravitino/pull/5848
### What changes were proposed in this pull request?
This PR adds the model catalog implementation.
### Why are the changes needed?
This is a part of work to support model manag