zrlw commented on code in PR #15633:
URL: https://github.com/apache/dubbo/pull/15633#discussion_r2280666464
##########
dubbo-registry/dubbo-registry-api/src/main/java/org/apache/dubbo/registry/client/DefaultServiceInstance.java:
##########
@@ -304,34 +305,86 @@ public boolean equals(Object o) {
if (!(o instanceof DefaultServiceInstance)) {
return false;
}
- DefaultServiceInstance that = (DefaultServiceInstance) o;
- boolean equals = Objects.equals(getServiceName(),
that.getServiceName())
- && Objects.equals(getHost(), that.getHost())
- && Objects.equals(getPort(), that.getPort());
- for (Map.Entry<String, String> entry : this.getMetadata().entrySet()) {
- if
(entry.getKey().equals(EXPORTED_SERVICES_REVISION_PROPERTY_NAME)) {
- continue;
- }
- if (entry.getValue() == null) {
- equals = equals && (entry.getValue() ==
that.getMetadata().get(entry.getKey()));
+ DefaultServiceInstance other = (DefaultServiceInstance) o;
+ if (!Objects.equals(serviceName, other.serviceName)
+ || !Objects.equals(host, other.host)
+ || !Objects.equals(port, other.port)) {
+ return false;
+ }
+
+ return compareMetadata(other.metadata);
+ }
+
+ /**
+ * Compares this instance's metadata with other instance's metadata for
equality.
+ * <p>
+ * The comparison is performed refer to the {@code equals} method of
{@code AbstractMap}. For each entry key:
+ * <ul>
+ * <li>if the key is EXPORTED_SERVICES_REVISION_PROPERTY_NAME or
TIMESTAMP_KEY,
+ * only the presence of the key is checked; their values are
ignored.</li>
+ * <li>For all other keys, both the key and its value must match.</li>
+ * </ul>
+ * <p>
+ * @param otherMetadata the other instance's metadata to compare with
+ * @return {@code true} if the metadata are considered equal according to
the above rules; {@code false} otherwise
+ */
+ private boolean compareMetadata(Map<String, String> otherMetadata) {
+ if (otherMetadata == this.metadata) {
+ return true;
+ }
+
+ if (otherMetadata.size() != this.metadata.size()) {
+ return false;
+ }
Review Comment:
compares size to ensure the metadata to compare with doesn't have extra keys.
--
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: notifications-unsubscr...@dubbo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@dubbo.apache.org
For additional commands, e-mail: notifications-h...@dubbo.apache.org
