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

    https://github.com/apache/spark/pull/22732#discussion_r225651483
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileFormatDataWriter.scala
 ---
    @@ -179,7 +179,8 @@ class DynamicPartitionDataWriter(
           val partitionName = ScalaUDF(
             ExternalCatalogUtils.getPartitionPathString _,
             StringType,
    -        Seq(Literal(c.name), Cast(c, StringType, 
Option(description.timeZoneId))))
    +        Seq(Literal(c.name), Cast(c, StringType, 
Option(description.timeZoneId))),
    +        Seq(false, false))
    --- End diff --
    
    ```suggestion
            handleNullForInputs = Seq(false, false))
    ```


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to