jerryshao commented on code in PR #7010:
URL: https://github.com/apache/gravitino/pull/7010#discussion_r2051930557
##########
api/src/main/java/org/apache/gravitino/model/ModelVersionChange.java:
##########
@@ -61,6 +61,16 @@ static ModelVersionChange removeProperty(String property) {
return new ModelVersionChange.RemoveProperty(property);
}
+ /**
+ * Create a ModelVersionChange for updating the uri of a model version.
+ *
+ * @param newUri The new uri to be set for the model version.
+ * @return A new ModelVersionChange instance for updating the uri of a model
version.
+ */
+ static ModelVersionChange updateUri(String newUri) {
Review Comment:
We should not allow to update to a null or empty URI, can you please add a
such check here or in the next PR?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
