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

    https://github.com/apache/carbondata/pull/2172#discussion_r181770149
  
    --- Diff: 
integration/spark-common/src/main/scala/org/apache/spark/util/PartitionUtils.scala
 ---
    @@ -223,4 +224,41 @@ object PartitionUtils {
           }
         }
       }
    +
    +  /**
    +   * Used to extract Partitioner Fields for aggregation datamaps.
    +   *
    +   */
    +  def getPartitionerFields(partitionColumn: Seq[String],
    --- End diff --
    
    Please add detail level comment for this method, 
    Condition for generating partition filed 


---

Reply via email to