Rename equals to baseEquals and hashCode to baseHashCode in JobInfo

mziccard · mziccard · commit e11a5aeffe77 · 2016-01-22T20:01:39.000+01:00
diff --git a/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/CopyJobInfo.java b/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/CopyJobInfo.java
@@ -178,12 +178,12 @@ ToStringHelper toStringHelper() {
 
   @Override
   public boolean equals(Object obj) {
-    return obj instanceof CopyJobInfo && Objects.equals(toPb(), ((CopyJobInfo) obj).toPb());
+    return obj instanceof CopyJobInfo && baseEquals((CopyJobInfo) obj);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(super.hashCode(), sourceTables, destinationTable, createDisposition,
+    return Objects.hash(baseHashCode(), sourceTables, destinationTable, createDisposition,
         writeDisposition);
   }
 
diff --git a/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/ExtractJobInfo.java b/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/ExtractJobInfo.java
@@ -212,12 +212,12 @@ ToStringHelper toStringHelper() {
 
   @Override
   public boolean equals(Object obj) {
-    return obj instanceof ExtractJobInfo && Objects.equals(toPb(), ((ExtractJobInfo) obj).toPb());
+    return obj instanceof ExtractJobInfo && baseEquals((ExtractJobInfo) obj);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(super.hashCode(), sourceTable, destinationUris, printHeader, fieldDelimiter,
+    return Objects.hash(baseHashCode(), sourceTable, destinationUris, printHeader, fieldDelimiter,
         format, compression);
   }
 
diff --git a/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/JobInfo.java b/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/JobInfo.java
@@ -264,14 +264,12 @@ public String toString() {
     return toStringHelper().toString();
   }
 
-  @Override
-  public int hashCode() {
+  protected final int baseHashCode() {
     return Objects.hash(jobId);
   }
 
-  @Override
-  public boolean equals(Object obj) {
-    return obj instanceof JobInfo && Objects.equals(toPb(), ((JobInfo) obj).toPb());
+  protected final boolean baseEquals(JobInfo jobInfo) {
+    return Objects.equals(toPb(), jobInfo.toPb());
   }
 
   Job toPb() {
diff --git a/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/LoadJobInfo.java b/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/LoadJobInfo.java
@@ -117,12 +117,12 @@ ToStringHelper toStringHelper() {
 
   @Override
   public boolean equals(Object obj) {
-    return obj instanceof LoadJobInfo && Objects.equals(toPb(), ((LoadJobInfo) obj).toPb());
+    return obj instanceof LoadJobInfo && baseEquals((LoadJobInfo) obj);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(super.hashCode(), sourceUris, configuration);
+    return Objects.hash(baseHashCode(), sourceUris, configuration);
   }
 
   @Override
diff --git a/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/QueryJobInfo.java b/gcloud-java-bigquery/src/main/java/com/google/gcloud/bigquery/QueryJobInfo.java
@@ -447,12 +447,12 @@ ToStringHelper toStringHelper() {
 
   @Override
   public boolean equals(Object obj) {
-    return obj instanceof QueryJobInfo && Objects.equals(toPb(), ((QueryJobInfo) obj).toPb());
+    return obj instanceof QueryJobInfo && baseEquals((QueryJobInfo) obj);
   }
 
   @Override
   public int hashCode() {
-    return Objects.hash(super.hashCode(), allowLargeResults, createDisposition, destinationTable,
+    return Objects.hash(baseHashCode(), allowLargeResults, createDisposition, destinationTable,
         defaultDataset, flattenResults, priority, query, tableDefinitions, useQueryCache,
         userDefinedFunctions, writeDisposition, dryRun);
   }

Original file line number	Diff line number	Diff line change
`@@ -178,12 +178,12 @@ ToStringHelper toStringHelper() {`
`178`	`178`
`179`	`179`	`@Override`
`180`	`180`	`public boolean equals(Object obj) {`
`181`		`- return obj instanceof CopyJobInfo && Objects.equals(toPb(), ((CopyJobInfo) obj).toPb());`
	`181`	`+ return obj instanceof CopyJobInfo && baseEquals((CopyJobInfo) obj);`
`182`	`182`	`}`
`183`	`183`
`184`	`184`	`@Override`
`185`	`185`	`public int hashCode() {`
`186`		`- return Objects.hash(super.hashCode(), sourceTables, destinationTable, createDisposition,`
	`186`	`+ return Objects.hash(baseHashCode(), sourceTables, destinationTable, createDisposition,`
`187`	`187`	`writeDisposition);`
`188`	`188`	`}`
`189`	`189`
Original file line number	Diff line number	Diff line change
`@@ -212,12 +212,12 @@ ToStringHelper toStringHelper() {`
`212`	`212`
`213`	`213`	`@Override`
`214`	`214`	`public boolean equals(Object obj) {`
`215`		`- return obj instanceof ExtractJobInfo && Objects.equals(toPb(), ((ExtractJobInfo) obj).toPb());`
	`215`	`+ return obj instanceof ExtractJobInfo && baseEquals((ExtractJobInfo) obj);`
`216`	`216`	`}`
`217`	`217`
`218`	`218`	`@Override`
`219`	`219`	`public int hashCode() {`
`220`		`- return Objects.hash(super.hashCode(), sourceTable, destinationUris, printHeader, fieldDelimiter,`
	`220`	`+ return Objects.hash(baseHashCode(), sourceTable, destinationUris, printHeader, fieldDelimiter,`
`221`	`221`	`format, compression);`
`222`	`222`	`}`
`223`	`223`
Original file line number	Diff line number	Diff line change
`@@ -264,14 +264,12 @@ public String toString() {`
`264`	`264`	`return toStringHelper().toString();`
`265`	`265`	`}`
`266`	`266`
`267`		`- @Override`
`268`		`- public int hashCode() {`
	`267`	`+ protected final int baseHashCode() {`
`269`	`268`	`return Objects.hash(jobId);`
`270`	`269`	`}`
`271`	`270`
`272`		`- @Override`
`273`		`- public boolean equals(Object obj) {`
`274`		`- return obj instanceof JobInfo && Objects.equals(toPb(), ((JobInfo) obj).toPb());`
	`271`	`+ protected final boolean baseEquals(JobInfo jobInfo) {`
	`272`	`+ return Objects.equals(toPb(), jobInfo.toPb());`
`275`	`273`	`}`
`276`	`274`
`277`	`275`	`Job toPb() {`
Original file line number	Diff line number	Diff line change
`@@ -117,12 +117,12 @@ ToStringHelper toStringHelper() {`
`117`	`117`
`118`	`118`	`@Override`
`119`	`119`	`public boolean equals(Object obj) {`
`120`		`- return obj instanceof LoadJobInfo && Objects.equals(toPb(), ((LoadJobInfo) obj).toPb());`
	`120`	`+ return obj instanceof LoadJobInfo && baseEquals((LoadJobInfo) obj);`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`@Override`
`124`	`124`	`public int hashCode() {`
`125`		`- return Objects.hash(super.hashCode(), sourceUris, configuration);`
	`125`	`+ return Objects.hash(baseHashCode(), sourceUris, configuration);`
`126`	`126`	`}`
`127`	`127`
`128`	`128`	`@Override`
Original file line number	Diff line number	Diff line change
`@@ -447,12 +447,12 @@ ToStringHelper toStringHelper() {`
`447`	`447`
`448`	`448`	`@Override`
`449`	`449`	`public boolean equals(Object obj) {`
`450`		`- return obj instanceof QueryJobInfo && Objects.equals(toPb(), ((QueryJobInfo) obj).toPb());`
	`450`	`+ return obj instanceof QueryJobInfo && baseEquals((QueryJobInfo) obj);`
`451`	`451`	`}`
`452`	`452`
`453`	`453`	`@Override`
`454`	`454`	`public int hashCode() {`
`455`		`- return Objects.hash(super.hashCode(), allowLargeResults, createDisposition, destinationTable,`
	`455`	`+ return Objects.hash(baseHashCode(), allowLargeResults, createDisposition, destinationTable,`
`456`	`456`	`defaultDataset, flattenResults, priority, query, tableDefinitions, useQueryCache,`
`457`	`457`	`userDefinedFunctions, writeDisposition, dryRun);`
`458`	`458`	`}`