jerqi commented on code in PR #6912:
URL: https://github.com/apache/gravitino/pull/6912#discussion_r2041359903
##########
api/src/main/java/org/apache/gravitino/model/ModelChange.java:
##########
@@ -97,4 +119,138 @@ public String toString() {
return "RenameModel " + newName;
}
}
+
+ /** A ModelChange to set a property and value of a model. */
+ final class SetProperty implements ModelChange {
+ private final String property;
+ private final String value;
+
+ /**
+ * Constructs a new {@link SetProperty} instance with the specified
property name and value.
+ *
+ * @param property The name of the property to be set.
+ * @param value The value to be set for the property.
+ */
+ public SetProperty(String property, String value) {
+ this.property = property;
+ this.value = value;
+ }
+
+ /**
+ * Retrieves the name of the property to be set.
+ *
+ * @return The name of the property to be set.
+ */
+ public String property() {
+ return property;
+ }
+
+ /**
+ * Retrieves the value to be set for the property.
+ *
+ * @return The value to be set for the property.
+ */
+ public String value() {
+ return value;
+ }
+
+ /**
+ * Compares this SetProperty instance with another object for equality.
Two instances are
+ * considered equal if they target the same property and set the same
value.
+ *
+ * @param obj The object to compare with this instance.
+ * @return {@code true} if the given object represents the same property
set; {@code false}
+ * otherwise.
+ */
+ @Override
+ public boolean equals(Object obj) {
+ if (obj == this) return true;
+ if (obj == null || getClass() != obj.getClass()) return false;
Review Comment:
You can use `obj instanceOf SetProperty ` directly. Null will return false.
It's more clear. This is the suggestion of the book `Effective Java`
--
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]
