Abyss-lord commented on code in PR #6967:
URL: https://github.com/apache/gravitino/pull/6967#discussion_r2046452515
##########
api/src/main/java/org/apache/gravitino/model/ModelVersionChange.java:
##########
@@ -34,13 +34,22 @@ public interface ModelVersionChange {
* Create a ModelVersionChange for updating the comment of a model version.
*
* @param newComment new comment to be set for the model version
- * @return a new ModelVersionChange instance for updating the comment of a
model version
+ * @return A new ModelVersionChange instance for updating the comment of a
model version
*/
static ModelVersionChange updateComment(String newComment) {
return new ModelVersionChange.UpdateComment(newComment);
}
- /** A ModelVersionChange to update the modelve version comment. */
+ /**
+ * Create a ModelVersionChange for removing the comment of a model version.
+ *
+ * @return A new ModelVersionChange instance for removing the comment of a
model version
+ */
+ static ModelVersionChange removeComment() {
Review Comment:
@jerryshao I see other types have Remove operations, like FileSet; should
they be consistent?
--
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]
