Address PR comments

Author: vikasrathee-cs

src/main/java/io/cdap/plugin/servicenow/apiclient/ServiceNowTableAPIClientImpl.java

@@ -79,13 +79,11 @@ public class ServiceNowTableAPIClientImpl extends RestAPIClient {
   private static final String GLIDE_DATE_TIME_DATATYPE = "glide_date_time";
   private static final Gson GSON = new Gson();
   private final ServiceNowConnectorConfig conf;
-  public final Boolean useConnection;
   public final SchemaType schemaType;
   public static JsonArray serviceNowJsonResultArray;
 
   public ServiceNowTableAPIClientImpl(ServiceNowConnectorConfig conf, Boolean useConnection) {
     this.conf = conf;
-    this.useConnection = useConnection;
     this.schemaType = getSchemaTypeBasedOnUseConnection(useConnection);
   }
 

src/main/java/io/cdap/plugin/servicenow/apiclient/ServiceNowTableAPIRequestBuilder.java

@@ -23,6 +23,7 @@
 
 import java.net.URLEncoder;
 import java.util.Arrays;
+import javax.annotation.Nullable;
 
 /**
  * ServiceNowTableAPIRequestBuilder.
@@ -51,6 +52,9 @@ public class ServiceNowTableAPIRequestBuilder extends RestAPIRequest.Builder {
 
   public ServiceNowTableAPIRequestBuilder(String instanceBaseUrl, String tableName, boolean isSchemaRequired,
                                           SchemaType schemaType) {
+    // if it is a schema call,  this URL value will get overridden by the later section.
+    this.setUrl(String.format(TABLE_API_URL_TEMPLATE, instanceBaseUrl, tableName));
+
     if (isSchemaRequired) {
       if (SchemaType.METADATA_API_BASED == schemaType) {
         this.setUrl(String.format(METADATA_API_URL_TEMPLATE, instanceBaseUrl, tableName));
@@ -60,8 +64,6 @@ public ServiceNowTableAPIRequestBuilder(String instanceBaseUrl, String tableName
         this.setUrl(String.format(TABLE_API_URL_TEMPLATE, instanceBaseUrl, tableName));
         this.setLimit(1);
       }
-    } else {
-      this.setUrl(String.format(TABLE_API_URL_TEMPLATE, instanceBaseUrl, tableName));
     }
   }