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

    https://github.com/apache/spark/pull/21883#discussion_r206414524
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
 ---
    @@ -284,6 +284,11 @@ object PartitioningUtils {
         }.mkString("/")
       }
     
    +  def getPathFragment(partitions: TablePartitionSpec): String = 
partitions.map {
    +    case (k, v) =>
    +      escapePathName(k) + "=" + escapePathName(v)
    +  }.mkString("/")
    --- End diff --
    
    TablePartitionSpec is a map. The output of this func might be different 
from the actual schema, right?
    
    For example, the partition schema is [partCol1, partCol2]. Your output 
could place partCol2 before partCol1


---

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

Reply via email to