dongjoon-hyun commented on code in PR #41428:
URL: https://github.com/apache/spark/pull/41428#discussion_r1214659820
##########
core/src/main/scala/org/apache/spark/internal/config/package.scala:
##########
@@ -2355,4 +2355,11 @@ package object config {
.version("3.3.0")
.intConf
.createWithDefault(5)
+
+ private[spark] val ALLOW_CUSTOM_CLASSPATH_BY_PROXY_USER_IN_CLUSTER_MODE =
+ ConfigBuilder("spark.submit.proxyUser.allowCustomClasspathInClusterMode")
+ .internal()
+ .version("3.3.3")
+ .booleanConf
+ .createWithDefault(false)
Review Comment:
I believe we can keep the existing behavior by having the default value
`true` here. Then, it removes the breaking change issue and allows some users
utilize this based on their situation. Could you change the default value,
@degant ?
##########
core/src/main/scala/org/apache/spark/internal/config/package.scala:
##########
@@ -2355,4 +2355,11 @@ package object config {
.version("3.3.0")
.intConf
.createWithDefault(5)
+
+ private[spark] val ALLOW_CUSTOM_CLASSPATH_BY_PROXY_USER_IN_CLUSTER_MODE =
+ ConfigBuilder("spark.submit.proxyUser.allowCustomClasspathInClusterMode")
+ .internal()
+ .version("3.3.3")
+ .booleanConf
+ .createWithDefault(false)
Review Comment:
I believe we can keep the existing behavior by having the default value
`true` here in branch-3.3. Then, it removes the breaking change issue and
allows some users utilize this based on their situation. Could you change the
default value, @degant ?
--
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]
