[GitHub] carbondata pull request #2571: [CARBONDATA-2792][schema restructure] Create ...

[kunal642]

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

    https://github.com/apache/carbondata/pull/2571#discussion_r206480372
  
    --- 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 --
    
    both filter conditions can be combined to one like .filter(a>2 && b<10).


---