srielau commented on code in PR #58549:
URL: https://github.com/apache/spark/pull/58549#discussion_r4002362243
##########
sql/core/src/main/scala/org/apache/spark/sql/classic/SparkSession.scala:
##########
@@ -887,11 +887,21 @@ class SparkSession private(
private[sql] def applySchemaToPythonRDD(
rdd: RDD[Array[Any]],
schema: StructType): DataFrame = {
+ val applyCharVarcharChecks =
Review Comment:
Follow-up in 4229ce348af and c8976436977: the shared cross-policy block
exposed two unrelated limitations (classic cannot build an encoder for a
retained CHAR output schema after switching to legacy mode, while Connect uses
a different local-relation path), so I removed that over-broad case. The
policy-capture regression is now classic-only: it creates under legacy
semantics, switches to standard semantics before the action, and verifies the
captured unchecked conversion remains in effect. The shared classic/Connect
test still covers standard padding and overflow plus legacy unchecked values.
--
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]
