Merge pull request #1398 from cloudsufi/patch/partition-field-validation

[PLUGIN-788] Fixed Partition Field Validation

Author: Pushpender Saini

src/main/java/io/cdap/plugin/gcp/bigquery/sink/BigQuerySinkConfig.java

@@ -410,6 +410,10 @@ private void validatePartitionProperties(@Nullable Schema schema, FailureCollect
     String tableName = getTable();
     String serviceAccount = getServiceAccount();
 
+    if (shouldCreatePartitionedTable()) {
+      validateColumnForPartition(partitionByField, schema, collector);
+    }
+
     if (project == null || dataset == null || tableName == null || serviceAccount == null) {
       return;
     }
@@ -435,11 +439,6 @@ private void validatePartitionProperties(@Nullable Schema schema, FailureCollect
       } else if (rangePartitioning != null) {
         validateRangePartitionTableWithInputConfiguration(table, rangePartitioning, collector);
       }
-      validateColumnForPartition(partitionByField, schema, collector);
-      return;
-    }
-    if (shouldCreatePartitionedTable()) {
-      validateColumnForPartition(partitionByField, schema, collector);
     }
   }
 
@@ -560,6 +559,14 @@ private void validateIntegerPartitioningRange(Long rangeStart, Long rangeEnd, Lo
         .withConfigProperty(NAME_RANGE_END);
     }
 
+    if (!containsMacro(NAME_RANGE_START) && !containsMacro(NAME_RANGE_END) && rangeStart != null && rangeEnd != null) {
+      if (rangeEnd <= rangeStart) {
+        collector.addFailure("Range End is less than or equal to Range Start.",
+                             "Range End must be greater than Range Start.")
+          .withConfigProperty(NAME_RANGE_END);
+      }
+    }
+
     if (!containsMacro(NAME_RANGE_INTERVAL)) {
       if (rangeInterval == null) {
         collector.addFailure(