You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by jackylk <gi...@git.apache.org> on 2018/03/21 11:22:44 UTC

[GitHub] carbondata pull request #2078: [CARBONDATA-2262] Support the syntax of 'STOR...

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

    https://github.com/apache/carbondata/pull/2078#discussion_r176048872
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSparkSqlParser.scala ---
    @@ -91,6 +91,9 @@ class CarbonHelperSqlAstBuilder(conf: SQLConf,
           case Some(value) =>
             if (value.children.get(1).getText.equalsIgnoreCase("by")) {
               value.storageHandler().STRING().getSymbol.getText
    +        } else if (value.children.get(1).getText.equalsIgnoreCase("as")
    --- End diff --
    
    can you call `value.children.get(1).getText` once and use the result


---