maxResults always > 0

Author: Neenu1995

java-bigquery/google-cloud-bigquery-jdbc/src/main/java/com/google/cloud/bigquery/jdbc/DataSource.java

@@ -671,8 +671,9 @@ public void setProjectId(String projectId) {
   }
 
   public void setMaxResults(Long maxResults) {
-    if (maxResults != null) {
-      validateNonNegative(maxResults, BigQueryJdbcUrlUtility.MAX_RESULTS_PROPERTY_NAME);
+    if (maxResults != null && maxResults <= 0) {
+      throw new BigQueryJdbcRuntimeException(
+          "Invalid value for MaxResults. It must be greater than 0.");
     }
     this.maxResults = maxResults;
   }

java-bigquery/google-cloud-bigquery-jdbc/src/test/java/com/google/cloud/bigquery/jdbc/BigQueryJdbcUrlUtilityTest.java

@@ -228,18 +228,16 @@ public void testInvalidConnectionProperties() {
 
     String url6 = "jdbc:bigquery://;RetryMaxDelay=-1";
     assertThrows(BigQueryJdbcRuntimeException.class, () -> DataSource.fromUrl(url6));
-  }
 
-  @Test
-  public void testValidZeroConnectionProperties() {
-    String url = "jdbc:bigquery://;MaxResults=0";
-    assertDoesNotThrow(() -> DataSource.fromUrl(url));
+    String url7 = "jdbc:bigquery://;MaxResults=0";
+    assertThrows(BigQueryJdbcRuntimeException.class, () -> DataSource.fromUrl(url7));
   }
 
   @Test
   public void testInvalidSetterValues() {
     DataSource ds = new DataSource();
     assertThrows(BigQueryJdbcRuntimeException.class, () -> ds.setMaxResults(-1L));
+    assertThrows(BigQueryJdbcRuntimeException.class, () -> ds.setMaxResults(0L));
     assertThrows(BigQueryJdbcRuntimeException.class, () -> ds.setTimeout(-1));
     assertThrows(BigQueryJdbcRuntimeException.class, () -> ds.setJobTimeout(-1));
     assertThrows(BigQueryJdbcRuntimeException.class, () -> ds.setRetryInitialDelay(-1));