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

    https://github.com/apache/carbondata/pull/2078#discussion_r176286537
  
    --- Diff: 
integration/spark2/src/main/spark2.1/org/apache/spark/sql/hive/CarbonSessionState.scala
 ---
    @@ -326,7 +326,8 @@ class CarbonSqlAstBuilder(conf: SQLConf, parser: 
CarbonSpark2SqlParser, sparkSes
         val fileStorage = helper.getFileStorage(ctx.createFileFormat)
     
         if (fileStorage.equalsIgnoreCase("'carbondata'") ||
    -        fileStorage.equalsIgnoreCase("'org.apache.carbondata.format'")) {
    +      fileStorage.equalsIgnoreCase("carbondata") ||
    --- End diff --
    
    It's different: 'carbondata' and carbondata.   
    
    carbon supported syntax "stored by 'carbondata'" before, but didn't  
supported  syntax "stored by carbondata".
    we need support syntax "STORED AS carbondata" like "STORED AS parquet"


---

Reply via email to