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

    https://github.com/apache/carbondata/pull/2571#discussion_r206503427
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/carbondata/spark/util/CarbonSparkUtil.scala
 ---
    @@ -87,18 +88,20 @@ object CarbonSparkUtil {
         val fields = new Array[String](
           carbonRelation.dimensionsAttr.size + carbonRelation.measureAttr.size)
         val carbonTable = carbonRelation.carbonTable
    +    val columnSchemas: mutable.Buffer[ColumnSchema] = 
carbonTable.getTableInfo.getFactTable.
    +      getListOfColumns.asScala.filter(_.getSchemaOrdinal != 
-1).filter(!_.isInvisible).
    --- End diff --
    
    @kunal642 change to 
      val columnSchemas: mutable.Buffer[ColumnSchema] = 
carbonTable.getTableInfo.getFactTable.
          getListOfColumns.asScala
          .filter(cSchema => cSchema.getSchemaOrdinal != -1 && 
!cSchema.isInvisible).
          sortWith(_.getSchemaOrdinal < _.getSchemaOrdinal)


---

Reply via email to