EamesTrinh commented on code in PR #58831:
URL: https://github.com/apache/spark/pull/58831#discussion_r4379710709
##########
sql/core/src/test/scala/org/apache/spark/sql/connector/InsertIntoTests.scala:
##########
@@ -736,6 +736,31 @@ trait InsertIntoSchemaEvolutionTests { this:
InsertIntoTests =>
}
}
+ Seq(
+ true -> Row(1, 2, 3),
+ false -> Row(2, 1, 3)
+ ).foreach { case (resolveNestedFieldsByPosition, expected) =>
+ test("Insert schema evolution: column list resolves nested fields by
position - " +
Review Comment:
Added collection coverage. The existing tests cover the default positional
behavior for arrays and map keys/values. This PR now also verifies the
flag-disabled compatibility behavior for arrays and maps too.
--
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]
