HyukjinKwon commented on a change in pull request #31421:
URL: https://github.com/apache/spark/pull/31421#discussion_r568382451
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -2472,6 +2472,16 @@ object SQLConf {
.booleanConf
.createWithDefault(true)
+ val LEGACY_PARSE_NULL_PARTITION_SPEC_AS_STRING_LITERAL =
+ buildConf("spark.sql.legacy.parseNullPartitionSpecAsStringLiteral")
+ .internal()
+ .doc("If it is set to true, a null partition value is parsed as a string
literal of its " +
+ "text representation, e.g., string 'null'. Otherwise, null partition
values are parsed " +
+ "as they are.")
+ .version("3.2.0")
Review comment:
should be 3.0.2 actually.
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -2472,6 +2472,16 @@ object SQLConf {
.booleanConf
.createWithDefault(true)
+ val LEGACY_PARSE_NULL_PARTITION_SPEC_AS_STRING_LITERAL =
+ buildConf("spark.sql.legacy.parseNullPartitionSpecAsStringLiteral")
+ .internal()
+ .doc("If it is set to true, a null partition value is parsed as a string
literal of its " +
+ "text representation, e.g., string 'null'. Otherwise, null partition
values are parsed " +
+ "as they are.")
+ .version("3.2.0")
Review comment:
I think we have documented only in 3.0.2 so far in such cases ..
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
