carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sounakr <...@git.apache.org>
Subject [GitHub] carbondata pull request #1595: [CARBONDATA-1868][Spark-2.2]Carbon-Spark2.2 I...
Date Thu, 07 Dec 2017 04:13:12 GMT
Github user sounakr commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1595#discussion_r155427042
  
    --- Diff: integration/spark2/src/main/spark2.2/CarbonSessionState.scala ---
    @@ -256,6 +292,57 @@ class CarbonSqlAstBuilder(conf: SQLConf, parser: CarbonSpark2SqlParser)
extends
         }
       }
     
    +  override def visitChangeColumn(ctx: ChangeColumnContext): LogicalPlan = {
    --- End diff --
    
    We can catch the command in strategy. But as AlterTableAddColumnsCommand is only added
in Spark 2.2. As our strategy is common for spark2.2. and spark2.1 it will be difficult to
match the case of AlterTableAddColumnsCommand. Currently the SparkSqlAstBuilder is separated
in 2.1 and 2.2 in CarbonSessionState.scala so it was easier to place it over there. 


---

Mime
View raw message