[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_r156958685
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
 ---
    @@ -932,6 +958,11 @@ abstract class CarbonDDLSqlParser extends 
AbstractCarbonSparkSQLParser {
         }
       }
     
    +  def isEscapeSequence(loadOption: String): Boolean = {
    --- End diff --
    
    Rename it to isValidEscapeSequence and also add one valid test  case for 
validating a correct escape sequence


---