Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1524#discussion_r151831633 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/load/ValidateUtil.scala --- @@ -17,35 +17,32 @@ package org.apache.carbondata.spark.load -import scala.collection.JavaConverters._ +import java.text.SimpleDateFormat -import org.apache.carbondata.core.constants.CarbonCommonConstants import org.apache.carbondata.core.metadata.schema.table.CarbonTable -import org.apache.carbondata.processing.loading.model.CarbonLoadModel import org.apache.carbondata.processing.loading.sort.SortScopeOptions import org.apache.carbondata.spark.exception.MalformedCarbonCommandException object ValidateUtil { - def validateDateFormat(dateFormat: String, table: CarbonTable, tableName: String): Unit = { - val dimensions = table.getDimensionByTableName(tableName).asScala + + /** + * validates both timestamp and date for illegal values --- End diff -- validate timestamp format and date format. please change
---