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

    https://github.com/apache/spark/pull/22357#discussion_r216218951
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetSchemaPruning.scala
 ---
    @@ -156,7 +161,7 @@ private[sql] object ParquetSchemaPruning extends 
Rule[LogicalPlan] {
         // in the resulting schema may differ from their ordering in the 
logical relation's
         // original schema
         val mergedSchema = requestedRootFields
    -      .map { case RootField(field, _) => StructType(Array(field)) }
    +      .map { case RootField(field, _, _) => StructType(Array(field)) }
    --- End diff --
    
    Not a big deal but `.map { root: RootField => StructType(Array(root.field)) 
}` per https://github.com/databricks/scala-style-guide#pattern-matching


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to