fix(bigquery): implement explicit equals in QueryRequestInfo

Implement explicit equals method in QueryRequestInfo to ensure
it matches hashCode (which includes jobTimeoutMs) and avoid
violating the equals/hashCode contract.
Also added unit tests for equals and hashCode.

TAG=agy
CONV=2773a7e8-a0fe-4d6f-89ce-762ad24aa78d

Author: lqiu96

java-bigquery/google-cloud-bigquery/src/main/java/com/google/cloud/bigquery/QueryRequestInfo.java

@@ -191,9 +191,28 @@ public int hashCode() {
 
   @Override
   public boolean equals(Object obj) {
-    return obj == this
-        || obj != null
-            && obj.getClass().equals(QueryRequestInfo.class)
-            && java.util.Objects.equals(toPb(), ((QueryRequestInfo) obj).toPb());
+    if (obj == this) {
+      return true;
+    }
+    if (obj == null || !obj.getClass().equals(QueryRequestInfo.class)) {
+      return false;
+    }
+    QueryRequestInfo other = (QueryRequestInfo) obj;
+    return java.util.Objects.equals(connectionProperties, other.connectionProperties)
+        && java.util.Objects.equals(defaultDataset, other.defaultDataset)
+        && java.util.Objects.equals(dryRun, other.dryRun)
+        && java.util.Objects.equals(labels, other.labels)
+        && java.util.Objects.equals(maximumBytesBilled, other.maximumBytesBilled)
+        && java.util.Objects.equals(maxResults, other.maxResults)
+        && java.util.Objects.equals(query, other.query)
+        && java.util.Objects.equals(queryParameters, other.queryParameters)
+        && java.util.Objects.equals(requestId, other.requestId)
+        && java.util.Objects.equals(createSession, other.createSession)
+        && java.util.Objects.equals(useQueryCache, other.useQueryCache)
+        && java.util.Objects.equals(useLegacySql, other.useLegacySql)
+        && java.util.Objects.equals(jobCreationMode, other.jobCreationMode)
+        && java.util.Objects.equals(formatOptions, other.formatOptions)
+        && java.util.Objects.equals(reservation, other.reservation)
+        && java.util.Objects.equals(jobTimeoutMs, other.jobTimeoutMs);
   }
 }

java-bigquery/google-cloud-bigquery/src/test/java/com/google/cloud/bigquery/QueryRequestInfoTest.java

@@ -180,6 +180,25 @@ public void testToPb() {
     assertEquals(TIMEOUT, requestWithTimeoutPb.getJobTimeoutMs());
   }
 
+  @Test
+  public void testEqualsAndHashCode() {
+    QueryRequestInfo info1 =
+        new QueryRequestInfo(
+            QUERY_JOB_CONFIGURATION_SUPPORTED, DataFormatOptions.newBuilder().build());
+    QueryRequestInfo info2 =
+        new QueryRequestInfo(
+            QUERY_JOB_CONFIGURATION_SUPPORTED, DataFormatOptions.newBuilder().build());
+
+    assertNotEquals(info1, info2);
+    assertNotEquals(info1.hashCode(), info2.hashCode());
+
+    assertEquals(info1, info1);
+    assertEquals(info1.hashCode(), info1.hashCode());
+
+    assertNotEquals(info1, null);
+    assertNotEquals(info1, "different type");
+  }
+
   @Test
   public void equalTo() {
     compareQueryRequestInfo(