jerryshao commented on code in PR #7881:
URL: https://github.com/apache/gravitino/pull/7881#discussion_r2252895392


##########
api/src/main/java/org/apache/gravitino/model/ModelVersionChange.java:
##########
@@ -302,6 +335,18 @@ final class UpdateUri implements ModelVersionChange {
      * @param newUri The new uri to be set for the model version.
      */
     public UpdateUri(String newUri) {
+      this.uriName = ModelVersion.URI_NAME_UNKNOWN;
+      this.newUri = newUri;
+    }
+
+    /**
+     * Creates a new {@link UpdateUri} instance with the specified new uri and 
its name.
+     *
+     * @param uriName The name of the uri to be updated for the model version.
+     * @param newUri The new uri to be set for the model version.
+     */
+    public UpdateUri(String uriName, String newUri) {
+      this.uriName = uriName;
       this.newUri = newUri;

Review Comment:
   This can be implemented as `this(xxx, xxx)`



-- 
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]

Reply via email to