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

[mohammadshahidkhan]

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

    https://github.com/apache/carbondata/pull/2571#discussion_r206525134
  
    --- 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  Changed the filter condition from && to ||, the earlier filters 
were serving the OR.


---