[GitHub] carbondata pull request #1653: [CARBONDATA-1893] Data load with multiple QUO...

[manishgupta88]

Github user manishgupta88 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1653#discussion_r157009066
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
 ---
    @@ -871,6 +871,32 @@ abstract class CarbonDDLSqlParser extends 
AbstractCarbonSparkSQLParser {
           throw new MalformedCarbonCommandException(errorMessage)
         }
     
    +    // Validate QUOTECHAR length
    +    if (options.exists(_._1.equalsIgnoreCase("QUOTECHAR"))) {
    +      val quoteChar: String = options.get("quotechar").get.head._2
    +      if (quoteChar.length > 1 && !isValidEscapeSequence(quoteChar)) {
    --- End diff --
    
    No need to validate quote character for a valid escape sequence


---