Annotate equals methods with Nullable in src/core/main

Author: vlsi

core/build.gradle.kts

@@ -56,6 +56,7 @@ dependencies {
     implementation("net.hydromatic:aggdesigner-algorithm")
     implementation("org.apache.commons:commons-dbcp2")
     implementation("org.apache.commons:commons-lang3")
+    implementation("org.checkerframework:checker-qual")
     implementation("commons-io:commons-io")
     implementation("org.codehaus.janino:commons-compiler")
     implementation("org.codehaus.janino:janino")

core/src/main/java/org/apache/calcite/interpreter/Row.java

@@ -16,6 +16,8 @@
  */
 package org.apache.calcite.interpreter;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Arrays;
 
 /**
@@ -64,7 +66,7 @@ public static Row of(Object...values) {
     return Arrays.hashCode(values);
   }
 
-  @Override public boolean equals(Object obj) {
+  @Override public boolean equals(@Nullable Object obj) {
     return obj == this
         || obj instanceof Row
         && Arrays.equals(values, ((Row) obj).values);

core/src/main/java/org/apache/calcite/jdbc/JavaRecordType.java

@@ -19,6 +19,8 @@
 import org.apache.calcite.rel.type.RelDataTypeField;
 import org.apache.calcite.rel.type.RelRecordType;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.List;
 import java.util.Objects;
 
@@ -37,7 +39,7 @@ public JavaRecordType(List<RelDataTypeField> fields, Class clazz) {
     this.clazz = Objects.requireNonNull(clazz);
   }
 
-  @Override public boolean equals(Object obj) {
+  @Override public boolean equals(@Nullable Object obj) {
     return this == obj
         || obj instanceof JavaRecordType
         && fieldList.equals(((JavaRecordType) obj).fieldList)

core/src/main/java/org/apache/calcite/materialize/Lattice.java

@@ -591,7 +591,7 @@ public int compareTo(@Nonnull Measure measure) {
       return Objects.hash(agg, args);
     }
 
-    @Override public boolean equals(Object obj) {
+    @Override public boolean equals(@Nullable Object obj) {
       return obj == this
           || obj instanceof Measure
           && this.agg.equals(((Measure) obj).agg)
@@ -664,7 +664,7 @@ public int compareTo(Column column) {
       return ordinal;
     }
 
-    @Override public boolean equals(Object obj) {
+    @Override public boolean equals(@Nullable Object obj) {
       return obj == this
           || obj instanceof Column
           && this.ordinal == ((Column) obj).ordinal;

core/src/main/java/org/apache/calcite/materialize/LatticeSuggester.java

@@ -632,7 +632,7 @@ public int hashCode() {
       return ordinalInQuery;
     }
 
-    public boolean equals(Object obj) {
+    public boolean equals(@Nullable Object obj) {
       return this == obj
           || obj instanceof TableRef
           && ordinalInQuery == ((TableRef) obj).ordinalInQuery;
@@ -658,7 +658,7 @@ private static class StepRef extends DefaultEdge {
       return ordinalInQuery;
     }
 
-    @Override public boolean equals(Object obj) {
+    @Override public boolean equals(@Nullable Object obj) {
       return this == obj
           || obj instanceof StepRef
           && ((StepRef) obj).ordinalInQuery == ordinalInQuery;

core/src/main/java/org/apache/calcite/materialize/LatticeTable.java

@@ -20,6 +20,8 @@
 import org.apache.calcite.rel.type.RelDataTypeField;
 import org.apache.calcite.util.Util;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.Objects;
 import javax.annotation.Nonnull;
 
@@ -37,7 +39,7 @@ public class LatticeTable {
     return t.getQualifiedName().hashCode();
   }
 
-  @Override public boolean equals(Object obj) {
+  @Override public boolean equals(@Nullable Object obj) {
     return this == obj
         || obj instanceof LatticeTable
         && t.getQualifiedName().equals(

core/src/main/java/org/apache/calcite/materialize/MaterializationActor.java

@@ -23,6 +23,8 @@
 import com.google.common.collect.HashMultimap;
 import com.google.common.collect.Multimap;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
@@ -97,7 +99,7 @@ static class QueryKey {
       this.path = path;
     }
 
-    @Override public boolean equals(Object obj) {
+    @Override public boolean equals(@Nullable Object obj) {
       return obj == this
           || obj instanceof QueryKey
           && sql.equals(((QueryKey) obj).sql)

core/src/main/java/org/apache/calcite/materialize/MaterializationKey.java

@@ -16,6 +16,8 @@
  */
 package org.apache.calcite.materialize;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.io.Serializable;
 import java.util.UUID;
 
@@ -32,7 +34,7 @@ public class MaterializationKey implements Serializable {
     return uuid.hashCode();
   }
 
-  @Override public boolean equals(Object obj) {
+  @Override public boolean equals(@Nullable Object obj) {
     return this == obj
         || obj instanceof MaterializationKey
         && uuid.equals(((MaterializationKey) obj).uuid);

core/src/main/java/org/apache/calcite/materialize/Path.java

@@ -18,6 +18,8 @@
 
 import com.google.common.collect.ImmutableList;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.List;
 
 /** A sequence of {@link Step}s from a root node (fact table) to another node
@@ -35,7 +37,7 @@ class Path {
     return id;
   }
 
-  @Override public boolean equals(Object obj) {
+  @Override public boolean equals(@Nullable Object obj) {
     return this == obj
         || obj instanceof Path
         && id == ((Path) obj).id;

core/src/main/java/org/apache/calcite/materialize/Step.java

@@ -24,6 +24,8 @@
 import com.google.common.collect.ImmutableList;
 import com.google.common.collect.Ordering;
 
+import org.checkerframework.checker.nullness.qual.Nullable;
+
 import java.util.List;
 import java.util.Objects;
 
@@ -68,7 +70,7 @@ static Step create(LatticeTable source, LatticeTable target,
     return Objects.hash(source, target, keys);
   }
 
-  @Override public boolean equals(Object obj) {
+  @Override public boolean equals(@Nullable Object obj) {
     return this == obj
         || obj instanceof Step
         && ((Step) obj).source.equals(source)