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

[jackylk]

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

    https://github.com/apache/carbondata/pull/1653#discussion_r157152181
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/CommonUtil.scala
 ---
    @@ -649,6 +650,25 @@ object CommonUtil {
         csvColumns
       }
     
    +  def isValidEscapeSequence(escapeChar: String): Boolean = {
    +    escapeChar.equals(NEW_LINE.getName) || 
escapeChar.equals(CARRIAGE_RETURN.getName) ||
    +    escapeChar.equals(TAB.getName) || escapeChar.equals(BACKSPACE.getName)
    +  }
    +
    +  def getEscapeChar(escapeCharacter: String): String = {
    --- End diff --
    
    move to a private function in CarbonLoadUtil


---