From db0d40938ba85b3a5c79f824178b44722f4489d0 Mon Sep 17 00:00:00 2001 From: cryptoe Date: Sat, 18 Jan 2025 11:14:54 +0530 Subject: [PATCH 1/2] Adding equals and hashcode impl --- .../apache/druid/server/security/Access.java | 22 +++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/server/src/main/java/org/apache/druid/server/security/Access.java b/server/src/main/java/org/apache/druid/server/security/Access.java index 8e3564032c89..349faffd0265 100644 --- a/server/src/main/java/org/apache/druid/server/security/Access.java +++ b/server/src/main/java/org/apache/druid/server/security/Access.java @@ -24,6 +24,7 @@ import org.apache.druid.query.policy.Policy; import javax.annotation.Nullable; +import java.util.Objects; import java.util.Optional; /** @@ -136,4 +137,25 @@ public String toString() return StringUtils.format("Allowed:%s, Message:%s, Policy: %s", allowed, message, policy); } + @Override + public boolean equals(Object o) + { + if (this == o) { + return true; + } + if (o == null || getClass() != o.getClass()) { + return false; + } + Access access = (Access) o; + return allowed == access.allowed && Objects.equals(message, access.message) && Objects.equals( + policy, + access.policy + ); + } + + @Override + public int hashCode() + { + return Objects.hash(allowed, message, policy); + } } From 74dbb17b715ee7c831ed9999564117fdf94e88fc Mon Sep 17 00:00:00 2001 From: cryptoe Date: Sun, 19 Jan 2025 23:37:14 +0530 Subject: [PATCH 2/2] Formatting --- .../main/java/org/apache/druid/server/security/Access.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/server/src/main/java/org/apache/druid/server/security/Access.java b/server/src/main/java/org/apache/druid/server/security/Access.java index 349faffd0265..117191568f66 100644 --- a/server/src/main/java/org/apache/druid/server/security/Access.java +++ b/server/src/main/java/org/apache/druid/server/security/Access.java @@ -147,10 +147,9 @@ public boolean equals(Object o) return false; } Access access = (Access) o; - return allowed == access.allowed && Objects.equals(message, access.message) && Objects.equals( - policy, - access.policy - ); + return allowed == access.allowed + && Objects.equals(message, access.message) + && Objects.equals(policy, access.policy); } @Override