mgaido91 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_r322602225
##########
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:
the reason why I created the properties object was to have it in a
synchronized block, but since we are in a lazy val it is probably useless...
----------------------------------------------------------------
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]
