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

    https://github.com/apache/carbondata/pull/2157#discussion_r181696200
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
 ---
    @@ -67,7 +67,10 @@ case class PreAggregateTableHelper(
         val partitionInfo = parentTable.getPartitionInfo
         val fields = fieldRelationMap.keySet.toSeq
         val tableProperties = mutable.Map[String, String]()
    -    val parentPartitionColumns = if (parentTable.isHivePartitionTable) {
    +    val usePartitioning = dataMapProperties.getOrDefault("partitioning", 
"true").toBoolean
    +    val parentPartitionColumns = if (!usePartitioning) {
    +      Seq()
    --- End diff --
    
    change to Seq.empty


---

Reply via email to