huaxingao commented on a change in pull request #33997:
URL: https://github.com/apache/spark/pull/33997#discussion_r708867061



##########
File path: 
sql/core/src/test/scala/org/apache/spark/sql/DataFrameWriterV2Suite.scala
##########
@@ -220,6 +220,33 @@ class DataFrameWriterV2Suite extends QueryTest with 
SharedSparkSession with Befo
       Seq(Row(1L, "a"), Row(2L, "b"), Row(4L, "d"), Row(5L, "e"), Row(6L, 
"f")))
   }
 
+  test("SPARK-36706 Overwrite: overwrite by expression: more than one 
filters") {

Review comment:
       Yes, the test would pass without the fix. but couldn't figure out an 
easy way to get the `OverwriteByExpressionExec` node and check the filters on 
it.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



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

Reply via email to