Added check of self-equals

alex268 · alex268 · commit 3b5ce15d8357 · 2025-05-20T13:50:57.000+01:00
diff --git a/topic/src/main/java/tech/ydb/topic/description/Consumer.java b/topic/src/main/java/tech/ydb/topic/description/Consumer.java
@@ -144,6 +144,9 @@ public Consumer build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/ConsumerDescription.java b/topic/src/main/java/tech/ydb/topic/description/ConsumerDescription.java
@@ -32,6 +32,9 @@ public List<ConsumerPartitionInfo> getPartitions() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/ConsumerPartitionInfo.java b/topic/src/main/java/tech/ydb/topic/description/ConsumerPartitionInfo.java
@@ -182,6 +182,9 @@ public int getConnectionNodeId() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/MultipleWindowsStat.java b/topic/src/main/java/tech/ydb/topic/description/MultipleWindowsStat.java
@@ -38,6 +38,9 @@ public long getPerDay() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/PartitionInfo.java b/topic/src/main/java/tech/ydb/topic/description/PartitionInfo.java
@@ -90,6 +90,9 @@ public PartitionInfo build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/PartitionStats.java b/topic/src/main/java/tech/ydb/topic/description/PartitionStats.java
@@ -130,6 +130,9 @@ public PartitionStats build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/SupportedCodecs.java b/topic/src/main/java/tech/ydb/topic/description/SupportedCodecs.java
@@ -51,6 +51,9 @@ public SupportedCodecs build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/TopicDescription.java b/topic/src/main/java/tech/ydb/topic/description/TopicDescription.java
@@ -183,6 +183,9 @@ public TopicDescription build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/description/TopicStats.java b/topic/src/main/java/tech/ydb/topic/description/TopicStats.java
@@ -80,6 +80,9 @@ public TopicStats build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/read/impl/OffsetsRangeImpl.java b/topic/src/main/java/tech/ydb/topic/read/impl/OffsetsRangeImpl.java
@@ -41,6 +41,9 @@ public void setEnd(long end) {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/settings/AutoPartitioningWriteStrategySettings.java b/topic/src/main/java/tech/ydb/topic/settings/AutoPartitioningWriteStrategySettings.java
@@ -73,6 +73,9 @@ public AutoPartitioningWriteStrategySettings build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }
diff --git a/topic/src/main/java/tech/ydb/topic/settings/PartitioningSettings.java b/topic/src/main/java/tech/ydb/topic/settings/PartitioningSettings.java
@@ -112,6 +112,9 @@ public PartitioningSettings build() {
 
     @Override
     public boolean equals(Object o) {
+        if (this == o) {
+            return true;
+        }
         if (o == null || getClass() != o.getClass()) {
             return false;
         }

Original file line number	Diff line number	Diff line change
`@@ -144,6 +144,9 @@ public Consumer build() {`
`144`	`144`
`145`	`145`	`@Override`
`146`	`146`	`public boolean equals(Object o) {`
	`147`	`+ if (this == o) {`
	`148`	`+ return true;`
	`149`	`+ }`
`147`	`150`	`if (o == null \|\| getClass() != o.getClass()) {`
`148`	`151`	`return false;`
`149`	`152`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,6 +32,9 @@ public List<ConsumerPartitionInfo> getPartitions() {`
`32`	`32`
`33`	`33`	`@Override`
`34`	`34`	`public boolean equals(Object o) {`
	`35`	`+ if (this == o) {`
	`36`	`+ return true;`
	`37`	`+ }`
`35`	`38`	`if (o == null \|\| getClass() != o.getClass()) {`
`36`	`39`	`return false;`
`37`	`40`	`}`
Original file line number	Diff line number	Diff line change
`@@ -182,6 +182,9 @@ public int getConnectionNodeId() {`
`182`	`182`
`183`	`183`	`@Override`
`184`	`184`	`public boolean equals(Object o) {`
	`185`	`+ if (this == o) {`
	`186`	`+ return true;`
	`187`	`+ }`
`185`	`188`	`if (o == null \|\| getClass() != o.getClass()) {`
`186`	`189`	`return false;`
`187`	`190`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,9 @@ public long getPerDay() {`
`38`	`38`
`39`	`39`	`@Override`
`40`	`40`	`public boolean equals(Object o) {`
	`41`	`+ if (this == o) {`
	`42`	`+ return true;`
	`43`	`+ }`
`41`	`44`	`if (o == null \|\| getClass() != o.getClass()) {`
`42`	`45`	`return false;`
`43`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -90,6 +90,9 @@ public PartitionInfo build() {`
`90`	`90`
`91`	`91`	`@Override`
`92`	`92`	`public boolean equals(Object o) {`
	`93`	`+ if (this == o) {`
	`94`	`+ return true;`
	`95`	`+ }`
`93`	`96`	`if (o == null \|\| getClass() != o.getClass()) {`
`94`	`97`	`return false;`
`95`	`98`	`}`
Original file line number	Diff line number	Diff line change
`@@ -130,6 +130,9 @@ public PartitionStats build() {`
`130`	`130`
`131`	`131`	`@Override`
`132`	`132`	`public boolean equals(Object o) {`
	`133`	`+ if (this == o) {`
	`134`	`+ return true;`
	`135`	`+ }`
`133`	`136`	`if (o == null \|\| getClass() != o.getClass()) {`
`134`	`137`	`return false;`
`135`	`138`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,9 @@ public SupportedCodecs build() {`
`51`	`51`
`52`	`52`	`@Override`
`53`	`53`	`public boolean equals(Object o) {`
	`54`	`+ if (this == o) {`
	`55`	`+ return true;`
	`56`	`+ }`
`54`	`57`	`if (o == null \|\| getClass() != o.getClass()) {`
`55`	`58`	`return false;`
`56`	`59`	`}`
Original file line number	Diff line number	Diff line change
`@@ -183,6 +183,9 @@ public TopicDescription build() {`
`183`	`183`
`184`	`184`	`@Override`
`185`	`185`	`public boolean equals(Object o) {`
	`186`	`+ if (this == o) {`
	`187`	`+ return true;`
	`188`	`+ }`
`186`	`189`	`if (o == null \|\| getClass() != o.getClass()) {`
`187`	`190`	`return false;`
`188`	`191`	`}`
Original file line number	Diff line number	Diff line change
`@@ -80,6 +80,9 @@ public TopicStats build() {`
`80`	`80`
`81`	`81`	`@Override`
`82`	`82`	`public boolean equals(Object o) {`
	`83`	`+ if (this == o) {`
	`84`	`+ return true;`
	`85`	`+ }`
`83`	`86`	`if (o == null \|\| getClass() != o.getClass()) {`
`84`	`87`	`return false;`
`85`	`88`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,9 @@ public void setEnd(long end) {`
`41`	`41`
`42`	`42`	`@Override`
`43`	`43`	`public boolean equals(Object o) {`
	`44`	`+ if (this == o) {`
	`45`	`+ return true;`
	`46`	`+ }`
`44`	`47`	`if (o == null \|\| getClass() != o.getClass()) {`
`45`	`48`	`return false;`
`46`	`49`	`}`