cloud-fan commented on a change in pull request #25738:
[SPARK-28939][SQL][FOLLOWUP] Fix JDK11 compilation due to ambiguous reference
URL: https://github.com/apache/spark/pull/25738#discussion_r322604537
##########
File path:
sql/core/src/main/scala/org/apache/spark/sql/execution/SQLExecutionRDD.scala
##########
@@ -39,7 +39,7 @@ class SQLExecutionRDD(
private val sqlConfigs = conf.getAllConfs
private lazy val sqlConfExecutorSide = {
val props = new Properties()
- props.putAll(sqlConfigs.asJava)
+ sqlConfigs.foreach { case (k, v) => props.setProperty(k, v) }
Review comment:
@mgaido91 can you send a followup PR to clean it up? Now we can't set the
SQL confs in a batch and I don't think it's useful to have this `Properties`.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
