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

    https://github.com/apache/carbondata/pull/1602#discussion_r154523178
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSparkSqlParser.scala
 ---
    @@ -18,6 +18,7 @@ package org.apache.spark.sql.parser
     
     import scala.collection.mutable
     
    +import org.antlr.v4.runtime.tree.TerminalNode
    --- End diff --
    
    No, I mean these type of validations to move to listeners, actually we no 
need to add these validations to carbon layer.  If other projects who want to 
block these operations can add their listener and just register it. I feel 
these configurations are not required.


---

Reply via email to