carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kumarvishal09 <...@git.apache.org>
Subject [GitHub] carbondata pull request #1682: [CARBONDATA-1910]do not allow tupleid, refere...
Date Wed, 20 Dec 2017 12:40:25 GMT
Github user kumarvishal09 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1682#discussion_r158012381
  
    --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSparkSqlParser.scala
---
    @@ -173,6 +174,18 @@ class CarbonHelperSqlAstBuilder(conf: SQLConf,
     
         // Ensuring whether no duplicate name is used in table definition
         val colNames = cols.map(_.name)
    +
    +    // do not allow below key words as column name
    +    colNames.foreach { col =>
    --- End diff --
    
    Please check scenario for CTAS... create table t1 as select sum(column1) as positionReference
from table. In this case name of the column in t1 will be positionReference. So in that case
also it should not allow to create carbon table. 


---

Mime
View raw message